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

PEP 8 love needed #80

Closed
dillongreen opened this issue Mar 8, 2011 · 1 comment
Closed

PEP 8 love needed #80

dillongreen opened this issue Mar 8, 2011 · 1 comment

Comments

@dillongreen
Copy link

there are a bunch of things in need for some PEP 8 love e.g.

sa@sub:~/0/python/django/django-extensions/django_extensions/management$ alias acsn; acsn pep8; pep8 *py
alias acsn='apt-cache search --names-only'
pep8 - Python PEP 8 code style checker
color.py:8:1: E302 expected 2 blank lines, found 1
jobs.py:40:1: W291 trailing whitespace
jobs.py:44:17: E225 missing whitespace around operator
jobs.py:75:80: E501 line too long (95 characters)
jobs.py:118:29: E261 at least two spaces before inline comment
modelviz.py:48:14: E401 multiple imports on one line
modelviz.py:159:27: E201 whitespace after '('
modelviz.py:180:25: E202 whitespace before '}'
modelviz.py:232:13: E303 too many blank lines (2)
modelviz.py:247:70: E231 missing whitespace after ','
sa@sub:~/0/python/django/django-extensions/django_extensions/management$ 
@trbs
Copy link
Member

trbs commented Mar 10, 2011

So very true, I've committed a large commit which should improve the situation.

Please take a look and reopen the ticket if you think this is not enough :)

(running pep8 with following cmdline: $ pep8 --ignore=E301,E202,E501,W602 $@ )

tehfink pushed a commit to tehfink/django-extensions that referenced this issue Apr 22, 2011
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

2 participants