Opened 5 weeks ago
Closed 2 weeks ago
#27072 closed enhancement (fixed)
py3: final fix for species
Reported by: | chapoton | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-8.7 |
Component: | python3 | Keywords: | |
Cc: | tscrim | Merged in: | |
Authors: | Frédéric Chapoton | Reviewers: | Travis Scrimshaw |
Report Upstream: | N/A | Work issues: | |
Branch: | aba9cd9 (Commits) | Commit: | aba9cd9754a3db1edb37bad7d41b12d046f1ada5 |
Dependencies: | Stopgaps: |
Description
Change History (14)
comment:1 Changed 5 weeks ago by
- Branch set to u/chapoton/27072
- Commit set to 0d4a59778dc4d04dcc728e219a2d60b8fe525835
- Status changed from new to needs_review
comment:2 Changed 5 weeks ago by
- Cc tscrim added
green bot (damn bot with no pyflakes..), please review
comment:3 Changed 5 weeks ago by
Perhaps it would be better to have min=0
be the default, and so we can change this test:
- if ((min is None and max is None) or - (max is None and self.get_aorder() >= min)): + if (max is None and (not min or self.get_aorder() >= min)): return self
comment:4 Changed 5 weeks ago by
- Commit changed from 0d4a59778dc4d04dcc728e219a2d60b8fe525835 to b579370f31ae0849b5c84b440df7d3b11fa344c6
Branch pushed to git repo; I updated commit sha1. New commits:
b579370 | fix
|
comment:5 Changed 5 weeks ago by
done, thanks
comment:6 Changed 5 weeks ago by
- Reviewers set to Travis Scrimshaw
- Status changed from needs_review to positive_review
Thank you.
comment:7 Changed 4 weeks ago by
- Status changed from positive_review to needs_work
Numerous doctest failures...
comment:8 Changed 3 weeks ago by
- Commit changed from b579370f31ae0849b5c84b440df7d3b11fa344c6 to acf0b7a6f205418177db45bc197b7bf47aa9afa9
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
acf0b7a | py3: final fix for species
|
comment:9 Changed 3 weeks ago by
- Status changed from needs_work to needs_review
indeed. Let us step back to my basic but working fix.
comment:10 Changed 3 weeks ago by
Dumb thing, but this change makes it overindented:
- (max is None and self.get_aorder() >= min)): - return self + (max is None and self.get_aorder() >= min)): + return self
Once fixed and green bot (either before or after), you can set back to a positive review.
comment:11 Changed 3 weeks ago by
- Commit changed from acf0b7a6f205418177db45bc197b7bf47aa9afa9 to aba9cd9754a3db1edb37bad7d41b12d046f1ada5
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
aba9cd9 | py3: final fix for species
|
comment:12 Changed 3 weeks ago by
right. Done. I will re-launch my bot on it.
comment:13 Changed 3 weeks ago by
- Status changed from needs_review to positive_review
ok, one green bot (the other is broken)
comment:14 Changed 2 weeks ago by
- Branch changed from u/chapoton/27072 to aba9cd9754a3db1edb37bad7d41b12d046f1ada5
- Resolution set to fixed
- Status changed from positive_review to closed
New commits:
py3: final fix for species