-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
New check: B040 #12097
Comments
Makes sense to me! |
note that I explicitly decided not to cover some thorny cases, but given you have the full logic for unused variables already, I think it might make sense for you to cover them. |
Hey! I've been using this crate for some time now and I'd love to contribute back! However, I'm a bit confused by the initial comment, what exactly are we trying to fix? |
We'd like to implement the new lint rule from upstream - this is basically a 'please port it' feature request! |
@Zac-HD Hi, I have added the rule that covers the same set of errors as one in upstream and even covers the unhandled cases there. Currently it only doesn't cover the reassignment of the exception like
|
PyCQA/flake8-bugbear#474 / PyCQA/flake8-bugbear#477
The text was updated successfully, but these errors were encountered: