Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
minor pep8 fix on every file#2460
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
pelson commentedSep 25, 2013
I don't necessarily agree. Adding parentheses is better IMHO: I'd like to get some other feedback on that before you have to re-write your |
mdboom commentedSep 25, 2013
Given the restriction that |
…achterstoo long according to pep8. This bulk-fixes every file in the library.
tacaswell commentedSep 25, 2013
Forcibly replaced the old commit with a new one using pelson's suggestion. Also caught a few files I missed last time. |
PEP8 fix on every file's from __future__ import
The fromfuture ... line added in#2226 is three charachters too long according to pep8. This bulk-fixes this by breaking the import up into 4 seperate lines.
No matter what every file will need to be touched again, and 4 separate imports seemed better style to me than any other way to break it up.
sedis a handy tool.