New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix pycodestyle E701/E702 in combinat #30514
Comments
Commit: |
Branch: u/chapoton/30514 |
New commits:
|
comment:2
may be you could also do changes like - j=(j+1)%(k+1)
+ j = (j + 1) % (k + 1) |
comment:3
J'ai essayé, autant que possible, de refrener mon envie de faire des changements pep8 tout autour. Ca augmente les risques de conflits de fusion, et c'est pas le but du ticket. |
Reviewer: David Coudert |
comment:4
OK. LGTM. |
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:
|
comment:6
just rebased for a conflict in tableau.py, setting back to positive |
Changed branch from u/chapoton/30514 to |
namely have one command per line
Component: refactoring
Author: Frédéric Chapoton
Branch/Commit:
1d3c652
Reviewer: David Coudert
Issue created by migration from https://trac.sagemath.org/ticket/30514
The text was updated successfully, but these errors were encountered: