Skip to content
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

E241 is more restrictive than PEP8 #4

Closed
florentx opened this issue Nov 21, 2009 · 1 comment
Closed

E241 is more restrictive than PEP8 #4

florentx opened this issue Nov 21, 2009 · 1 comment

Comments

@florentx
Copy link
Contributor

I was looking at some Python code and I had a second though about E241.
This check is about "multiple spaces after comma". It is not part of PEP8.

According to PEP 8, we should avoid "More than one space around an assignment (or other) operator to align it with another."
This is enforced by E221,E222, E223 and E224.

But the comma is a delimiter, not an operator. It binds less strongly than an operator.
Sometimes it may be a good idea to use extraneous whitespace for readability.

Additionally, I found this obvious example in the Python language reference.
http://docs.python.org/dev/py3k/reference/lexical_analysis.html#implicit-line-joining

month_names = ['Januari', 'Februari', 'Maart',      # These are the
               'April',   'Mei',      'Juni',       # Dutch names
               'Juli',    'Augustus', 'September',  # for the months
               'Oktober', 'November', 'December']   # of the year

So, I propose to lift the check E241.

@florentx
Copy link
Contributor Author

florentx commented Mar 4, 2010

Done in 0.5.0

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant