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

Fix PEP-8 warnings #1

Merged
merged 1 commit into from Jun 19, 2012

Conversation

Projects
None yet
2 participants
@dpursehouse
Copy link
Contributor

commented Jun 19, 2012

Fixed the following warnings:

E301 expected 1 blank line, found 0
E127 continuation line over-indented for visual indent

Which were found by running:

pep8.py --max-line-length 80

(PEP-8 checker pep8.py from github.com/jcrocholl/pep8)

Fix PEP-8 warnings
Fixed the following warnings:

    E301 expected 1 blank line, found 0
    E127 continuation line over-indented for visual indent

Which were found by running:

    pep8.py --max-line-length 80

(PEP-8 checker pep8.py from github.com/jcrocholl/pep8)
@keleshev

This comment has been minimized.

Copy link
Contributor

commented Jun 19, 2012

A bit of a shame that pep257 had pep8 warnings—so I'm pulling this :-)

keleshev added a commit that referenced this pull request Jun 19, 2012

@keleshev keleshev merged commit 5688296 into PyCQA:master Jun 19, 2012

@dpursehouse

This comment has been minimized.

Copy link
Contributor Author

commented Jun 20, 2012

Thanks!

By the way, I've added this project on Ohloh:

https://www.ohloh.net/p/pep257

@keleshev

This comment has been minimized.

Copy link
Contributor

commented Jun 20, 2012

@dpursehouse never heard of ohloh before—looks interesting.

What do you think of pep257? I heard people say it's too strict, and is following the PEP too literally.

@dpursehouse

This comment has been minimized.

Copy link
Contributor Author

commented Jun 20, 2012

I haven't used it in depth yet, but it looks good. IMO stricter is better, but then I'm quite the pedant :)

Nurdok pushed a commit that referenced this pull request Mar 25, 2015

Merge pull request #1 from GreenSteam/master
Bump version to 0.5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.