-
-
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
[changelog] Create separate entries for reimported/shadowed-import #7903
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.
Thanks!
doc/whatsnew/fragments/4836.newcheck
Outdated
@@ -0,0 +1,3 @@ | |||
Add a ``shadowed-import`` message for aliased imports. |
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.
The file name should probably be new_check
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.
I added a small check for this, the result is : doc/whatsnew/fragments/4836.newchekc should probably be named '4836.new_check' instead.
or doc/whatsnew/fragments/4836.glofbzid must be named one of '4836.internal', '4836.removed_check', '4836.extension', '4836.bugfix', '4836.other', '4836.new_check', '4836.false_negative', '4836.false_positive' instead.
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! This will definitely help. It has been an issue for me a couple of times as well.
3289012
to
cf9416a
Compare
Pull Request Test Coverage Report for Build 3622995218
π - Coveralls |
cf9416a
to
c93d53f
Compare
c93d53f
to
13ad92e
Compare
doc/whatsnew/fragments/4836.newcheck
Outdated
@@ -0,0 +1,3 @@ | |||
Add a ``shadowed-import`` message for aliased imports. |
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! This will definitely help. It has been an issue for me a couple of times as well.
Type of Changes
Description
See #7756 (comment)