x/vmware-nsx
Code Issues Proposed changes

Enable hacking H233 rule

Browse Source 
H233: Check that all occurrences look like print functions, not print
operator.

Change-Id: I4600eb32e1a8ecba13fc252d7ef08681672a40a7
This commit is contained in:
ZhiQiang Fan 2014-01-23 09:31:27 +08:00
parent a2220fad2c
commit bb932df665
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
neutron/wsgi.py
View File

@ -941,13 +941,13 @@ class Debug(Middleware):
print(("*" * 40) + " REQUEST ENVIRON") print(("*" * 40) + " REQUEST ENVIRON")
for key, value in req.environ.items(): for key, value in req.environ.items():
print(key, "=", value) print(key, "=", value)
print print()
resp = req.get_response(self.application) resp = req.get_response(self.application)
print(("*" * 40) + " RESPONSE HEADERS") print(("*" * 40) + " RESPONSE HEADERS")
for (key, value) in resp.headers.iteritems(): for (key, value) in resp.headers.iteritems():
print(key, "=", value) print(key, "=", value)
print print()
resp.app_iter = self.print_generator(resp.app_iter) resp.app_iter = self.print_generator(resp.app_iter)
@ -961,7 +961,7 @@ class Debug(Middleware):
sys.stdout.write(part) sys.stdout.write(part)
sys.stdout.flush() sys.stdout.flush()
yield part yield part
print print()
class Router(object): class Router(object):

3
tox.ini
View File

@ -35,11 +35,10 @@ commands = {posargs}
# E711/E712 comparison to False should be 'if cond is False:' or 'if not cond:' # E711/E712 comparison to False should be 'if cond is False:' or 'if not cond:'
# query = query.filter(Component.disabled == False) # query = query.filter(Component.disabled == False)
# E125 continuation line does not distinguish itself from next logical line # E125 continuation line does not distinguish itself from next logical line
# H233 Python 3.x incompatible use of print operator
# H301 one import per line # H301 one import per line
# H302 import only modules # H302 import only modules
# TODO(marun) H404 multi line docstring should start with a summary # TODO(marun) H404 multi line docstring should start with a summary
ignore = E711,E712,E125,H233,H301,H302,H404 ignore = E711,E712,E125,H301,H302,H404
show-source = true show-source = true
builtins = _ builtins = _
exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,tools exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,tools