-
-
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
Merging of use-implicit-booleaness
-like checkers for consistency and performance
#6870
Merging of use-implicit-booleaness
-like checkers for consistency and performance
#6870
Conversation
6626fc8
to
30aa974
Compare
Pull Request Test Coverage Report for Build 3440302019
π - Coveralls |
This comment has been minimized.
This comment has been minimized.
30aa974
to
e89dedc
Compare
Following #7629 we can merge everything in the same checker but still make one check optional by default. |
First step prior doing pylint-dev#6870 in order to make it more reviewable.
First step prior doing pylint-dev#6870 in order to make it more reviewable.
First step prior doing pylint-dev#6870 in order to make it more reviewable.
β¦codebase As it will become a default check in pylint-dev#6870 it will be easier to review it if we separate the two
This make the message more understandable and permit to make pylint-dev#6870 more reviewable
Originally done in order to normalize the use-implicit-booleaness check. This make the message more understandable and permit to make pylint-dev#6870 more reviewable.
β¦codebase As it will become a default check in pylint-dev#6870 it will be easier to review it if we separate the two
012e78a
to
f18d4ee
Compare
use-implicit-booleaness
-like checkersuse-implicit-booleaness
-like checkers
β¦v#8592) The extension `compare-to-empty-string` is only applicable when the expression being compared is strictly a `str`. The extension `compare-to-zero` is only applicable when the expression being compared is strictly an `int`.
f18d4ee
to
eb0931c
Compare
This comment has been minimized.
This comment has been minimized.
eb0931c
to
7aeb690
Compare
This comment has been minimized.
This comment has been minimized.
There might be some optimization between |
use-implicit-booleaness
-like checkersuse-implicit-booleaness
-like checkers for consistency and performance
This comment has been minimized.
This comment has been minimized.
β¦ated typing hint?)
48a4a15
to
616a234
Compare
π€ Effect of this PR on checked open source code: π€ Effect on astroid:
The following messages are no longer emitted:
Effect on black:
The following messages are no longer emitted:
Effect on django:
The following messages are no longer emitted:
This comment was truncated because GitHub allows only 65536 characters in a comment. This comment was generated for commit 616a234 |
Type of Changes
Description
Follow-up to #8630
Closes #6871
Closes #5234