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

Upgrade linters to the latest version #438

Merged
merged 13 commits into from
Jun 5, 2021

Conversation

cjolowicz
Copy link
Collaborator

@cjolowicz cjolowicz commented Jun 3, 2021

Bump the dependencies of Nox's lint session:

  • Bump black from 19.3b0 to 21.5b2
  • Bump flake8 from 3.7.8 to 3.9.2
  • Bump isort from 4.3.21 to 5.8.0
  • Bump mypy from 0.720 to 0.812

Fix new warnings and type errors.

Apply changes in code style and import ordering.

Use the new profile feature of isort 5 to achieve compatibility with black.

Supercedes #435
Supercedes #436
Supercedes #437

Copy link
Collaborator

@theacodes theacodes left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for doing this!

force_grid_wrap=0
use_parentheses=True
line_length=88
profile=black
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice. :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes :)

@cjolowicz cjolowicz merged commit 41b9c79 into wntrblm:main Jun 5, 2021
@cjolowicz cjolowicz deleted the upgrade-linters branch June 5, 2021 08:50
@cjolowicz cjolowicz mentioned this pull request Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants