-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
flake8 upgrade to 3.8.3 #1759
flake8 upgrade to 3.8.3 #1759
Conversation
setup.py
Outdated
extras_require['docs'] + | ||
extras_require['dev'] | ||
extras_require['tester'] | ||
+ extras_require['linter'] # noqa: W503 |
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.
I would be ok with adding W503
to the ignored warnings in our tox.ini
file.
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.
yeah. right! will do
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.
Thanks for doing this cleanup for us! Would you be up for adding W503 to the ignored linting codes so we can drop a few of the # noqa: W503
comments?
(also, you should work on getting CI passing) |
some of the CI will fail because of Known issue with Infura IPFS endpoint |
cffa3ef
to
1464074
Compare
@pratik-vii thanks for taking this on! I just pulled in the most recent master that should have the tests fixed, so as soon as those pass, I'll pull this in. 🎉 |
a5d2f44
to
b88a419
Compare
What was wrong?
Related to Issue #1720
How was it fixed?
Cute Animal Picture