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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closes #1684 #1850
Closes #1684 #1850
Conversation
@@ -11,7 +11,7 @@ | |||
from wemake_python_styleguide.logic import walk | |||
from wemake_python_styleguide.logic.naming import name_nodes | |||
from wemake_python_styleguide.logic.nodes import get_parent | |||
from wemake_python_styleguide.logic.tree import keywords, operators | |||
from wemake_python_styleguide.logic.tree import keywords, operators, variables |
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.
[pep8] reported by reviewdog 馃惗
F401 'wemake_python_styleguide.logic.tree.variables' imported but unused
Codecov Report
@@ Coverage Diff @@
## master #1850 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 107 107
Lines 5991 5992 +1
Branches 1341 1343 +2
=========================================
+ Hits 5991 5992 +1
Continue to review full report at Codecov.
|
"""Checks if unpacking target is correct.""" | ||
if isinstance(target, ast.Tuple): | ||
return all( | ||
_is_valid_single(element) is not None |
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.
_is_valid_single
returns bool
so why the result is compared to None
?
Shouldn't it be just _is_valid_single(element)
?
I have made things!
Checklist
CHANGELOG.md
Related issues
馃檹 Please, if you or your company is finding wemake-python-styleguide valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/wemake-python-styleguide. As a thank you, your profile/company logo will be added to our main README which receives hundreds of unique visitors per day.