From edca0c2a3d5282ba3d8b1ebe2d138ed8e84a9853 Mon Sep 17 00:00:00 2001 From: Sean Dague Date: Tue, 8 Jan 2013 17:09:07 -0500 Subject: make runtests -p act more like tox now that all the N402 fixes are in, start enforcing it Change-Id: I9d495849bf698e5af142d0ccafe551ac0e8a600c --- run_tests.sh | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) (limited to 'run_tests.sh') diff --git a/run_tests.sh b/run_tests.sh index c4a1d9efc..a79c8564c 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -116,18 +116,24 @@ function run_pep8 { srcfiles=`find nova -type f -name "*.py" ! -wholename "nova\/openstack*"` srcfiles+=" `find bin -type f ! -name "nova.conf*" ! -name "*api-paste.ini*"`" srcfiles+=" `find tools -type f -name "*.py"`" + srcfiles+=" `find plugins -type f -name "*.py"`" + srcfiles+=" `find smoketests -type f -name "*.py"`" srcfiles+=" setup.py" # Until all these issues get fixed, ignore. - ignore='--ignore=N402,E12,E711,E721,E712' + ignore='--ignore=E12,E711,E721,E712' ${wrapper} python tools/hacking.py ${ignore} ${srcfiles} + # NOTE(sdague): as of grizzly-2 these are passing however leaving the comment + # in here in case we need to break it out when we get more of our hacking working + # again. + # # NOTE(sirp): Dom0 plugins are written for Python 2.4, meaning some HACKING # checks are too strict. - pep8onlyfiles=`find plugins -type f -name "*.py"` - pep8onlyfiles+=" `find plugins/xenserver/xenapi/etc/xapi.d/plugins/ -type f -perm +111`" - ${wrapper} pep8 ${ignore} ${pep8onlyfiles} + # pep8onlyfiles=`find plugins -type f -name "*.py"` + # pep8onlyfiles+=" `find plugins/xenserver/xenapi/etc/xapi.d/plugins/ -type f -perm +111`" + # ${wrapper} pep8 ${ignore} ${pep8onlyfiles} } -- cgit