-
Notifications
You must be signed in to change notification settings - Fork 5
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
#164243636 Implement pep8 standards on codebase #11
Conversation
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
da05689
to
43b2368
Compare
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.
0 violations and 0 reported. Looks good
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.
Everything looks okay
43b2368
to
8cda689
Compare
install flake8 and create .flake8 config file - add flake8 script to .travis.yml - fix pep8 errors on codebase [Finishes #164243636]
8cda689
to
5ce5c6c
Compare
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
What does this PR do?
Adds testing of code for PEP8 style guide compliance
Description of Task to be completed?
How should this be manually tested?
git clone -b origin/ch-implement-pep8-164243636 https://github.com/andela/ah-legion-backend.git
pip install -r requirements.txt
from the root folder.find ./authors/ -name \*.py -exec flake8 -v {} +
Any background context you want to provide?
PEP8 styled code is highly readable and easier to maintain
What are the relevant pivotal tracker stories?
#164243636