From 7c9af1293790a1e78f9d9b4e91d3babd0be6d01c Mon Sep 17 00:00:00 2001 From: David Peraza Date: Tue, 7 Jan 2014 10:38:55 -0600 Subject: [PATCH] Check for pep8 E226 and E24 By default, pep8.py ignores E226 and E24. By changing tox.ini to not have an ignore clause, it started using the default. Since we were explicitely ignoring things until recently, E226 and E24 where being checked. This patch adds an ignore clause to tox.ini and fixes some E226 errors to match our pep8 checks to other projects in openstack. Change-Id: Ia0bc0e6bcb4b4c6e54d61bae2fd7cad48303a62a --- ceilometer/event/converter.py | 4 ++-- tox.ini | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/ceilometer/event/converter.py b/ceilometer/event/converter.py index 5557acb30..e064126f7 100644 --- a/ceilometer/event/converter.py +++ b/ceilometer/event/converter.py @@ -375,8 +375,8 @@ def setup_events(trait_plugin_mgr): errmsg = (_("Invalid YAML syntax in Event Definitions file " "%(file)s at line: %(line)s, column: %(column)s.") % dict(file=config_file, - line=mark.line+1, - column=mark.column+1)) + line=mark.line + 1, + column=mark.column + 1)) else: errmsg = (_("YAML error reading Event Definitions file " "%(file)s") diff --git a/tox.ini b/tox.ini index 4f1e3d300..1e7445830 100644 --- a/tox.ini +++ b/tox.ini @@ -43,6 +43,7 @@ deps = -r{toxinidir}/requirements.txt commands = {posargs} [flake8] +ignore = None builtins = _ exclude=.venv,.git,.tox,dist,doc,./ceilometer/openstack/common,*lib/python*,*egg,tools,nova_tests,build show-source = True