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

Switch to flake8 linter and fix warnings #59

Merged
merged 3 commits into from Oct 16, 2017

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Contributor

commented Oct 16, 2017

The flake8 linter is a wrapper around pep8 and pyflakes. In addition to checking for PEP style compliance, flake8 will also find bugs with an essentially zero false positive rate.

As such, it's well worth running the flake8 linter instead of pep8. A full listing of the issues that flake will detect can be found here.

c-w added some commits Oct 16, 2017

Switch to flake8 linter
The flake8 linter is a wrapper around pep8 and pyflakes. In addition to
checking for PEP style compliance, flake8 will also find bugs with an
essentially zero false positive rate. As such, it's well worth running
the flake8 linter instead of pep8. A full listing of the issues that
flake will detect can be found here:
http://flake8.pycqa.org/en/latest/user/error-codes.html

@DaKnOb DaKnOb merged commit 2987c46 into DaKnOb:master Oct 16, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
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.