diff options
author | Steven Dake <sdake@redhat.com> | 2013-02-27 10:26:46 -0700 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2013-02-28 20:18:01 +0000 |
commit | 637740a4f1aa64e192c05638b76ad2355909e346 (patch) | |
tree | 33b42fd8e54b95db0725d0a3f15e4c477f1e6640 /run_tests.sh | |
parent | 52fc9ac812ed89798c9ef008065fc58eb3ea94db (diff) | |
download | heat-637740a4f1aa64e192c05638b76ad2355909e346.tar.gz |
Update infrastructure to more closely match other OpenStack projects
This patch imports nova's HACKING.rst guidelines altered for Heat.
This patch imports nova's run_pep8.sh with some minor alterations.
This patch imports nova's hacking.py extensions to pep8.
This patch places the run_pep8 logic in a separate file rather than
having it spread between tox.ini and run_tests.sh. The HACKING.rst
requirements as well as general pep8 requirements are then enforced
with run_pep8.sh.
Avoid running once the N802 rule until dependency changelogs are
less then 50 characters.
Change-Id: If46c14668eae3cf11b8fbff46d766cc589d701aa
Fixes: Bug #1131275
Diffstat (limited to 'run_tests.sh')
-rwxr-xr-x | run_tests.sh | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/run_tests.sh b/run_tests.sh index feda1934d..8a5dbc981 100755 --- a/run_tests.sh +++ b/run_tests.sh @@ -46,10 +46,8 @@ function run_tests { } function run_pep8 { - echo "Running pep8..." - PEP8_OPTIONS="--exclude=$PEP8_EXCLUDE --repeat" - PEP8_INCLUDE="bin/heat-cfn bin/heat-boto bin/heat-api-cfn bin/heat-api bin/heat-engine heat tools setup.py heat/testing/runner.py" - ${wrapper} pep8 $PEP8_OPTIONS $PEP8_INCLUDE + echo "Running PEP8 and HACKING compliance check..." + bash -c "${wrapper} tools/run_pep8.sh" } # run unit tests with pep8 when no arguments are specified |