openstack/deb-swift
Code Issues Proposed changes

Merge "pep8: Don't override '_' symbol"

Browse Source
This commit is contained in:
Jenkins 2015-08-26 19:51:54 +00:00 committed by Gerrit Code Review
commit f0af259aae
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
swift/common/memcached.py
View File

@ -443,7 +443,7 @@ class MemcacheRing(object):
with Timeout(self._io_timeout): with Timeout(self._io_timeout):
sock.sendall(msg) sock.sendall(msg)
# Wait for the set to complete # Wait for the set to complete
for _ in range(len(mapping)): for line in range(len(mapping)):
fp.readline() fp.readline()
self._return_conn(server, fp, sock) self._return_conn(server, fp, sock)
return return

3
tox.ini
View File

@ -55,7 +55,6 @@ commands = bandit -c bandit.yaml -r swift bin -n 5 -p gate
[flake8] [flake8]
# it's not a bug that we aren't using all of hacking, ignore: # it's not a bug that we aren't using all of hacking, ignore:
# F402: import '_' from line 51 shadowed by loop variable
# F812: list comprehension redefines ... # F812: list comprehension redefines ...
# H101: Use TODO(NAME) # H101: Use TODO(NAME)
# H202: assertRaises Exception too broad # H202: assertRaises Exception too broad
@ -69,6 +68,6 @@ commands = bandit -c bandit.yaml -r swift bin -n 5 -p gate
# H405: multi line docstring summary not separated with an empty line # H405: multi line docstring summary not separated with an empty line
# H501: Do not use self.__dict__ for string formatting # H501: Do not use self.__dict__ for string formatting
# H703: Multiple positional placeholders # H703: Multiple positional placeholders
ignore = F402,F812,H101,H202,H233,H234,H301,H306,H401,H403,H404,H405,H501,H703 ignore = F812,H101,H202,H233,H234,H301,H306,H401,H403,H404,H405,H501,H703
exclude = .venv,.tox,dist,doc,*egg exclude = .venv,.tox,dist,doc,*egg
show-source = True show-source = True