-
-
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
Improve output of consider-using-generator
message for min()
calls with default
keyword
#8582
Improve output of consider-using-generator
message for min()
calls with default
keyword
#8582
Conversation
…s with `default` keyword.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #8582 +/- ##
=======================================
Coverage 95.91% 95.91%
=======================================
Files 174 174
Lines 18371 18394 +23
=======================================
+ Hits 17620 17643 +23
Misses 751 751
|
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.
LGTM, should we backport this kind of bug fixes ?
🤖 Effect of this PR on checked open source code: 🤖 Effect on django:
The following messages are no longer emitted:
This comment was generated for commit aec217f |
Why not. |
…s with `default` keyword (#8582) (#8583) (cherry picked from commit 4a485e2) Co-authored-by: Jacob Walls <[email protected]>
Type of Changes
Description
Closes #8563