-
-
Notifications
You must be signed in to change notification settings - Fork 519
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 pycodestyle E741 errors (#663) #671
Conversation
…, in this case, 'l's) and remove ignore of this error in tox.ini
…, in this case, 'l's) and remove ignore of this error in tox.ini
Codecov Report
@@ Coverage Diff @@
## master #671 +/- ##
==========================================
- Coverage 93.86% 90.64% -3.23%
==========================================
Files 11 11
Lines 2364 2362 -2
Branches 0 395 +395
==========================================
- Hits 2219 2141 -78
Misses 145 145
- Partials 0 76 +76
Continue to review full report at Codecov.
|
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.
One tiny thing to fix, otherwise looks good -- thanks for the PR!
tox/result.py
Outdated
@@ -63,8 +63,8 @@ def set_python_info(self, pythonexecutable): | |||
version=version) | |||
|
|||
def get_commandlog(self, name): | |||
l = self.dict.setdefault(name, []) | |||
return CommandLog(self, l) | |||
list = self.dict.setdefault(name, []) |
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.
list
shadows the builtin list
function, I'd pick a different name here
Fix pycodestyle E741 errors and re-introduce testing for this in tox. In all these cases, it was
l
and it represented alist
. This is not a good name, so I tried to get something more descriptive. In one case, it was passed to a constructor which hadlist
as an argument, so I decided to stick with that.This fixes #663.