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

fix compatibility with flake8 > 3.7 #68

Merged
merged 1 commit into from
May 12, 2020

Conversation

noviluni
Copy link

@noviluni noviluni commented May 11, 2020

flake8==3.8 has been released and it deprecates some functions, causing this package to stop working as expected and generating a lot of errors.

I fixed it to work with this new version.

It would be really nice to release this as soon as possible to avoid some pipelines to fail.

Let me know what you think @tholo 😄

Fixes #66

@noviluni noviluni force-pushed the fix_new_flake8_version_compatibility branch from d0c9aee to 1da98fc Compare May 11, 2020 22:16
@noviluni noviluni force-pushed the fix_new_flake8_version_compatibility branch from 1da98fc to e2461d1 Compare May 11, 2020 22:17
@hoIIer
Copy link

hoIIer commented May 11, 2020

+1

@tholo tholo merged commit de9840c into tholo:master May 12, 2020
cottsay added a commit to ros-infrastructure/ros_buildfarm that referenced this pull request May 26, 2020
Modeled after tholo/pytest-flake8#68.
Also resolve flake8 violations after latest flake8 release.
spenczar added a commit to spenczar/strimzi-registry-operator that referenced this pull request Oct 26, 2021
pytest-flake8 uses private APIs of flake8. One of these APIs was removed in
flake8 3.8, which temporarily broke pytest-flake8. This was fixed in
pytest-flake8 v1.0.6.

References:

Github Issue describing the bug:
tholo/pytest-flake8#66

Fix: tholo/pytest-flake8#68

Release tag that includes fix:
https://github.com/tholo/pytest-flake8/releases/tag/1.0.6
@Et7f3
Copy link

Et7f3 commented Apr 17, 2023

And it broken again with flake6 :(

@hoIIer
Copy link

hoIIer commented Apr 17, 2023

@Et7f3 I ditched this package and just use flake8 directly now since it works with latest pytest versions.

# lint.
flake8 .

# test.
pytest .

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.

failes with upcoming flake8 3.8.0
6 participants