summaryrefslogtreecommitdiff
path: root/run_tests.sh
diff options
context:
space:
mode:
authorSteven Dake <sdake@redhat.com>2013-02-27 10:26:46 -0700
committerGerrit Code Review <review@openstack.org>2013-02-28 20:18:01 +0000
commit637740a4f1aa64e192c05638b76ad2355909e346 (patch)
tree33b42fd8e54b95db0725d0a3f15e4c477f1e6640 /run_tests.sh
parent52fc9ac812ed89798c9ef008065fc58eb3ea94db (diff)
downloadheat-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-xrun_tests.sh6
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