-
-
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
Fix unused-import
to checkdummy-variables-rgx
#8566
Conversation
for more information, see https://pre-commit.ci
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #8566 +/- ##
==========================================
- Coverage 95.91% 95.91% -0.01%
==========================================
Files 174 174
Lines 18370 18369 -1
==========================================
- Hits 17619 17618 -1
Misses 751 751
|
for more information, see https://pre-commit.ci
This comment has been minimized.
This comment has been minimized.
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.
Pretty good first contribution, thank you ! This also fixed a false positive in pylint which is why the pylint check is failing :)
That's what I figured as well! |
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.
Reviewed
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.
π
Resolve #8500 Co-authored-by: Pierre Sassoulas <[email protected]> (cherry picked from commit 0cd41b1)
π€ Effect of this PR on checked open source code: π€ Effect on psycopg:
Effect on pytest:
This comment was generated for commit 53a8696 |
Resolve #8500 Co-authored-by: Pierre Sassoulas <[email protected]> (cherry picked from commit 0cd41b1) Co-authored-by: RSTdefg <[email protected]>
Type of Changes
Description
Fixed
unused-import
so that it observes thedummy-variables-rgx
option. Also added one testcase to test this behavior.Closes #8500