-
Notifications
You must be signed in to change notification settings - Fork 39
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
Update add_options for flake8 6.0 #111
Conversation
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.
LGTM
The new 6.0.0 contains a number of backwards incompatible changes for which plugins need to be updated and configuration needs to be updated. Sadly the maintainer no longer accepts contributions or discussion See PyCQA/flake8#1760
When is this expected to be released? |
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.
Looks good to me - Tested with Flake8 v5 & v6 by running the unittests and by invoking the cli with various options to check they work as expected.
The number of reviewers seems to be too low ATM, it might need more approvals? |
ty for the merge |
@zheller would you mind making a new release please 🙏 |
@zheller make a release |
@evrial demanding things from an OSS contributor like a petulant child is uncalled for. |
@dopry fair enough. what options do we have? fork and call flake8-quotes2 ? |
|
This seems to make
flake8-quotes
work withflake8
6 for me, i.e. tests pass and projects lint without crashes. For detailshttps://flake8.pycqa.org/en/latest/release-notes/6.0.0.html#backwards-incompatible-changes
Fixes #110