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

Explicitly state no compatibility with flake8 >= 5.0.0 #970

Merged
merged 1 commit into from
Aug 11, 2022

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented Aug 1, 2022

The latest release of flake8 drops API compatibility. We should eventually add support for the newer flake8, but this should keep CI running in the mean time.

@cottsay cottsay added the bug label Aug 1, 2022
@cottsay cottsay self-assigned this Aug 1, 2022
The pycodestyle and pyflakes requirements added here are to deal with
incompatibilities introduced in newer versions of those flake8 plugins
with the older flake8 requirement of this package.
Copy link
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

This looks generally reasonable to me. I'm wondering about the addition of pycodestyle; can you explain more about it?

@cottsay
Copy link
Member Author

cottsay commented Aug 11, 2022

I'm wondering about the addition of pycodestyle; can you explain more about it?

From an offline conversation: there are dependency couplings with the newer flake8 that pip doesn't seem to want to resolve for us, so it was necessary to pin that as well even though it's not a direct dependency.

@cottsay cottsay merged commit b274edb into master Aug 11, 2022
@cottsay cottsay deleted the cottsay/flake8 branch August 11, 2022 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants