-
Notifications
You must be signed in to change notification settings - Fork 35
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
ci: Replace pep8 by flake8 and separate linting stage on Travis #162
Conversation
2 similar comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great. Thanks.
LGTM.
.travis.yml
Outdated
@@ -8,7 +8,7 @@ python: | |||
- "pypy" | |||
- "pypy3" | |||
install: "pip install -r requirements/core.txt;pip install -r requirements/test.txt" | |||
before_script: "pep8" | |||
before_script: skip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit. This can be removed right?
.travis.yml
Outdated
@@ -17,12 +17,21 @@ after_success: | |||
|
|||
# Integration tests need to run first to reset the PR build status to pending |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change this comment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Summary
This implements flake8 which is a wrapper over pyflakes, pycodestyle(PEP8), and circular complexity checks.
Also, the linting stage has been separated so that it immediately fails the build rather than creating separate environments for multiple python versions and failing all the jobs for the same reason.
Test plan
Issues