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

Added flake8 #58

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added flake8 #58

wants to merge 4 commits into from

Conversation

Satyabrat35
Copy link
Contributor

  1. I have added flake8 for linting.
  2. In travis.yml, i have added --exit-zero to ensure that errors and warnings like white-space trailing, indentation, code line length etc. are ignored (caused the build to fail). In case you want to have white-space warnings to be added, you can just remove that bit of code.
    Review Test Travis #57

@ritwik12
Copy link
Owner

ritwik12 commented Feb 6, 2020

@Satyabrat35 It's better to have warnings like white-space trailing, indentation, code line length etc

@Satyabrat35
Copy link
Contributor Author

Okay,I will fix it.

@ritwik12
Copy link
Owner

ritwik12 commented Feb 6, 2020

Thanks @Satyabrat35 Use autopep8 or yapf :)

@ritwik12
Copy link
Owner

ritwik12 commented Jul 3, 2021

@Satyabrat35 Ping, reminder for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants