-
Notifications
You must be signed in to change notification settings - Fork 6
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: B028 got renamed to B907. See link in the description #507
base: release/1.4
Are you sure you want to change the base?
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.
Hmm... I'm on the fence whether to change all these warnings now or wait for the next major version to do it. While it is a bugfix, it is a breaking change since changing the stacklevel
to 2
will cause certain warnings to change in existing code (and potentially on any other CI that uses pylake
).
Updating this would be nice.
I wonder if doing something similar as Basically, they find the first stack level that isn't in pandas and use that one. |
065d28f
to
97479d4
Compare
97479d4
to
0eb6fe0
Compare
415e510
to
6cf6907
Compare
0eb6fe0
to
08d3719
Compare
b6b76a8
to
2971af5
Compare
2971af5
to
177fef5
Compare
177fef5
to
424fa8e
Compare
424fa8e
to
48bea15
Compare
48bea15
to
ea89a78
Compare
PyCQA/flake8-bugbear#333
This PR adjusts the ignore rule of flake8, accordingly.