From d93fa17563a835c472467f7f77cbfcbd236289bf Mon Sep 17 00:00:00 2001 From: Jesse Zoldak Date: Tue, 7 Oct 2014 10:27:10 -0400 Subject: [PATCH] Lower the pep8 threshold for failing builds --- scripts/all-tests.sh | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/scripts/all-tests.sh b/scripts/all-tests.sh index 3aa5a97c4b..417ab443c7 100755 --- a/scripts/all-tests.sh +++ b/scripts/all-tests.sh @@ -55,6 +55,10 @@ set -e # ############################################################################### +# Violations thresholds for failing the build +PYLINT_THRESHOLD=4800 +PEP8_THRESHOLD=675 + source $HOME/jenkins_env # Clean up previous builds @@ -103,8 +107,8 @@ SHARD=${SHARD:="all"} case "$TEST_SUITE" in "quality") - paver run_pep8 -l 725 > pep8.log || { cat pep8.log; EXIT=1; } - paver run_pylint -l 4800 > pylint.log || { cat pylint.log; EXIT=1; } + paver run_pep8 -l $PEP8_THRESHOLD > pep8.log || { cat pep8.log; EXIT=1; } + paver run_pylint -l $PYLINT_THRESHOLD > pylint.log || { cat pylint.log; EXIT=1; } paver run_quality # Need to create an empty test result so the post-build