From 48a0fdc49e1da915abe6d751e4705f5f6174c367 Mon Sep 17 00:00:00 2001 From: Chris Yeoh Date: Wed, 20 Nov 2013 22:35:08 +1030 Subject: [PATCH] Don't gate on E125 E125 overreaches on what pep8 specifies. See https://github.com/jcrocholl/pep8/issues/126 In practice E125 can be a pain as compliance requires manual indenting if using emacs and refactoring tools don't always get it right. There's a bit of a discussion about it here (on a tempest changeset): https://review.openstack.org/#/c/36788/ Change-Id: Ic73ab3c4a47f33de9145e0c7db2d8674230c2fe0 --- tox.ini | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tox.ini b/tox.ini index f3e720e62c17..53e10c8e4436 100644 --- a/tox.ini +++ b/tox.ini @@ -48,7 +48,8 @@ commands = {posargs} # TODO Hacking 0.6 checks to fix # H102 Apache 2.0 license header not found -ignore = E121,E122,E123,E124,E126,E127,E128,E711,E712,H102,H404,F403,F811,F841,H803 +# E125 is deliberately excluded. See https://github.com/jcrocholl/pep8/issues/126 +ignore = E121,E122,E123,E124,E125,E126,E127,E128,E711,E712,H102,H404,F403,F811,F841,H803 exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools [hacking]