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

flake8 upgrade to 3.8.3 #1759

Merged
merged 1 commit into from
Oct 12, 2020
Merged

flake8 upgrade to 3.8.3 #1759

merged 1 commit into from
Oct 12, 2020

Conversation

pratik-vii
Copy link
Contributor

@pratik-vii pratik-vii commented Oct 2, 2020

What was wrong?

Related to Issue #1720

How was it fixed?

  • upgraded flake8 to 3.8.3
  • W503 ignored where required.

Cute Animal Picture

Koala!!

@pratik-vii pratik-vii marked this pull request as draft October 2, 2020 10:51
@pratik-vii pratik-vii marked this pull request as ready for review October 2, 2020 11:05
setup.py Outdated
extras_require['docs'] +
extras_require['dev']
extras_require['tester']
+ extras_require['linter'] # noqa: W503
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah. right! will do

Copy link
Member

@pipermerriam pipermerriam left a 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?

@pipermerriam
Copy link
Member

(also, you should work on getting CI passing)

@pratik-vii
Copy link
Contributor Author

(also, you should work on getting CI passing)

some of the CI will fail because of Known issue with Infura IPFS endpoint

#1757 (comment)

@kclowes
Copy link
Collaborator

kclowes commented Oct 12, 2020

@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. 🎉

@kclowes kclowes merged commit 7fb3141 into ethereum:master Oct 12, 2020
@kclowes kclowes mentioned this pull request Oct 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants