Skip to content

Commit 8041539

Browse files
committed
don't ignore rules that are already enforced
H302 / H304 are clean in the neutron source tree, so it seems like they shouldn't be ignored. This requires no code changes to enable. If there is a longer standing reason for these ignores, would be great to document in tox.ini. Change-Id: Ib0f483e4f8fbd328815775a2a6f58be448172bd9
1 parent 9f6e1b6 commit 8041539

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

tox.ini

+1-3
Original file line numberDiff line numberDiff line change
@@ -52,16 +52,14 @@ commands = {posargs}
5252
# F812 list comprehension redefines name from line
5353
# H104 file contains nothing but comments
5454
# H237 module is removed in Python 3
55-
# H302 import only modules
56-
# H304 no relative imports
5755
# H305 imports not grouped correctly
5856
# H307 like imports should be grouped together
5957
# H401 docstring should not start with a space
6058
# H402 one line docstring needs punctuation
6159
# H405 multi line docstring summary not separated with an empty line
6260
# H904 Wrap long lines in parentheses instead of a backslash
6361
# TODO(marun) H404 multi line docstring should start with a summary
64-
ignore = E125,E126,E128,E129,E251,E265,E713,F402,F811,F812,H104,H237,H302,H304,H305,H307,H401,H402,H404,H405,H904
62+
ignore = E125,E126,E128,E129,E251,E265,E713,F402,F811,F812,H104,H237,H305,H307,H401,H402,H404,H405,H904
6563
show-source = true
6664
builtins = _
6765
exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools,.ropeproject

0 commit comments

Comments
 (0)