-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 #5370: Emit redefined-outer-name
when a nested except handler shadows an outer one
#5630
Fix #5370: Emit redefined-outer-name
when a nested except handler shadows an outer one
#5630
Conversation
… handler shadows an outer one.
Pull Request Test Coverage Report for Build 1649860076
💛 - Coveralls |
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.
Already looks good, thank you @jacobtylerwalls !
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.
Thanks once again @jacobtylerwalls!
I feel like you have a New Years resolution of leaving no used-before-assignment
and redefined-outer-name
unfixed in 2022 😄
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com> Co-authored-by: Daniël van Noord <13665637+DanielNoord@users.noreply.github.com>
for more information, see https://pre-commit.ci
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.
Perfect, thanks ! (I guess this is also going to be an inspiration for fixing #4391)
Type of Changes
Description
Emit
redefined-outer-name
when a nested except handler shadows (overwrites) the name used in an outer except handler.Implementation borrows from the
redefined-outer-name
implementation for if branches.Closes #5370, Closes #4434