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

flake8: B028 got renamed to B907. See link in the description #507

Open
wants to merge 3 commits into
base: release/1.4
Choose a base branch
from

Conversation

tobiasjj
Copy link
Member

@tobiasjj tobiasjj commented May 10, 2023

PyCQA/flake8-bugbear#333

This PR adjusts the ignore rule of flake8, accordingly.

Copy link
Member

@JoepVanlier JoepVanlier left a 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.

@JoepVanlier JoepVanlier added the enhancement New feature or request label May 15, 2023
@JoepVanlier
Copy link
Member

I wonder if doing something similar as pandas does would make sense for this: https://github.com/pandas-dev/pandas/blob/49851bbb723f758bbdbd983fe0cb36c2bc5d1b83/pandas/util/_exceptions.py#L30

Basically, they find the first stack level that isn't in pandas and use that one.

@JoepVanlier JoepVanlier force-pushed the tobiasjj/fix_flake8 branch 9 times, most recently from 065d28f to 97479d4 Compare August 8, 2023 13:43
@JoepVanlier JoepVanlier changed the base branch from main to reformat_tests August 8, 2023 14:02
Base automatically changed from reformat_tests to main August 8, 2023 14:48
@JoepVanlier JoepVanlier force-pushed the tobiasjj/fix_flake8 branch 2 times, most recently from b6b76a8 to 2971af5 Compare August 15, 2023 14:01
@JoepVanlier JoepVanlier changed the base branch from main to release/1.4 March 4, 2024 09:21
@JoepVanlier JoepVanlier changed the base branch from release/1.4 to close_tiffs March 4, 2024 09:52
Base automatically changed from close_tiffs to release/1.4 March 4, 2024 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants