Enable H407,H305,H307,E122 ignore E123
After bumping the hacking version to the 0.9.x series ignores were
added for several rules. This commit fixes the violations for a subset
of these rules and re-enables the checks.
E123 is ignored because this rule is ignored in the default pep8 by
default, due to a lack of consensus around it.
Change-Id: I8f14cd2ca6afc38d3fe8ee758272071111022896
diff --git a/tox.ini b/tox.ini
index 9c32121..88a8cfa 100644
--- a/tox.ini
+++ b/tox.ini
@@ -109,7 +109,8 @@
[flake8]
# E125 is a won't fix until https://github.com/jcrocholl/pep8/issues/126 is resolved. For further detail see https://review.openstack.org/#/c/36788/
# H402 skipped because some docstrings aren't sentences
-# Skipped because of new hacking 0.9: H407,H405,H904,H305,E123,H307,E122,E129,E128
-ignore = E125,H402,H404,H407,H405,H904,H305,E123,H307,E122,E129,E128
+# E123 skipped because it is ignored by default in the default pep8
+# Skipped because of new hacking 0.9: H405,H904,E129,E128
+ignore = E125,H402,E123,H404,H405,H904,E129,E128
show-source = True
exclude = .git,.venv,.tox,dist,doc,openstack,*egg