From cfd29196fedf41dcd61d0df6b0109dc8e43abfc8 Mon Sep 17 00:00:00 2001 From: LiuNanke Date: Thu, 14 Jan 2016 16:45:59 +0800 Subject: [PATCH] Clean up removed hacking rule from [flake8] ignore lists We bump hacking>=0.10.2, and hacking removed some rules, for the full list of rules please see [1]. So don't need them any more. Hacking related commits: Remove H904 in commit b1fe19ebebe47a36b905d709467f5e82521bbd96 Remove H803 in commit f01ce4fd822546cbd52a0aedc49184bddbfe1b10 Remove H307 in commit ec4833b206c23b0b6f9c6b101c70ab925a5e9c67 Remove H305 in commit 8f1fcbdb9aa4fc61349e5e879153c722195b1233 [1]https://github.com/openstack-dev/hacking/blob/master/setup.cfg#L30 Change-Id: I24b82c1913d3d42cc5228b1db700b787623fcdc5 --- test-requirements.txt | 2 +- tox.ini | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/test-requirements.txt b/test-requirements.txt index a50a202..5e4c304 100644 --- a/test-requirements.txt +++ b/test-requirements.txt @@ -2,7 +2,7 @@ # of appearance. Changing the order has an impact on the overall integration # process, which may cause wedges in the gate later. -hacking>=0.9.2,<0.10 +hacking>=0.10.2,<0.11 # Apache-2.0 coverage>=3.6 extras diff --git a/tox.ini b/tox.ini index 617831d..c3f42d6 100644 --- a/tox.ini +++ b/tox.ini @@ -32,10 +32,10 @@ commands = commands = sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html [flake8] -# H803 skipped on purpose per list discussion. # E123, E125 skipped as they are invalid PEP-8. show-source = True -ignore = E123,E125,H803 +ignore = E123,E125 builtins = _ exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,releasenotes/source/conf.py +