Browse Source

Remove E12* from list of deliberately ignored flake8 rules

I367064ecaa6d1fd9d918f7ce003303e2db660647  Added E12 to the
ignored on purpose list, which also makes no sense E12 was never meant
to be skipped all together (I was the one who added E12 to the flake8
ignore list (Ifc8924914b5a0d625bc8df6442ee85eb21459cde)

Then I4cf246e3ec932ba0d2391eb8bcb793b28b005b4c came along and updated
the list to break out the E12 ignored on purpose list.

Drop all E12 rules from the ignored on purpose list, as we do not want
to ignore these on purpose.

Change-Id: Ic400047347a82aebbc0f6ce6f5061e7757a02d3d
Joe Gordon 4 years ago
parent
commit
1adc6d0251
1 changed files with 1 additions and 2 deletions
  1. 1
    2
      tox.ini

+ 1
- 2
tox.ini View File

@@ -41,8 +41,7 @@ downloadcache = ~/cache/pip
41 41
 # H904 wrap long lines in parentheses instead of a backslash
42 42
 # reason: removed in hacking (https://review.openstack.org/#/c/101701/)
43 43
 #
44
-# Additional checks are also ignored on purpose: E124, E126,
45
-# E127, E128, E129, F811, F821
44
+# Additional checks are also ignored on purpose: F811, F821
46 45
 ignore = E124,E126,E127,E128,E129,F811,F821,H402,H404,H405,H904
47 46
 show-source = True
48 47
 exclude=.venv,.git,.tox,dist,*openstack/common*,*lib/python*,*egg,build,doc/source/conf.py

Loading…
Cancel
Save