Enhance Jenkins integration of safe template linting
- Add rule based thresholds - Add safecommit to Jenkins
This commit is contained in:
@@ -13,7 +13,7 @@ set -e
|
||||
# Violations thresholds for failing the build
|
||||
export PYLINT_THRESHOLD=4175
|
||||
export JSHINT_THRESHOLD=7550
|
||||
export SAFELINT_THRESHOLD=2700
|
||||
source scripts/safelint_thresholds.sh
|
||||
|
||||
doCheckVars() {
|
||||
if [ -n "$CIRCLECI" ] ; then
|
||||
|
||||
@@ -85,7 +85,9 @@ case "$TEST_SUITE" in
|
||||
echo "Running code complexity report (python)."
|
||||
paver run_complexity > reports/code_complexity.log || echo "Unable to calculate code complexity. Ignoring error."
|
||||
echo "Running safe template linter report."
|
||||
paver run_safelint -l $SAFELINT_THRESHOLD > safelint.log || { cat safelint.log; EXIT=1; }
|
||||
paver run_safelint -t $SAFELINT_THRESHOLDS > safelint.log || { cat safelint.log; EXIT=1; }
|
||||
echo "Running safe commit linter report."
|
||||
paver run_safecommit_report > safecommit.log || { cat safecommit.log; EXIT=1; }
|
||||
# Run quality task. Pass in the 'fail-under' percentage to diff-quality
|
||||
echo "Running diff quality."
|
||||
paver run_quality -p 100 || EXIT=1
|
||||
|
||||
47
scripts/safelint_thresholds.sh
Executable file
47
scripts/safelint_thresholds.sh
Executable file
@@ -0,0 +1,47 @@
|
||||
#!/usr/bin/env bash
|
||||
set -e
|
||||
|
||||
###############################################################################
|
||||
#
|
||||
# safelint_thresholds.sh
|
||||
#
|
||||
# The thresholds used for paver run_safelint when used with various CI
|
||||
# systems.
|
||||
#
|
||||
###############################################################################
|
||||
|
||||
# Violations thresholds for failing the build
|
||||
export SAFELINT_THRESHOLDS='
|
||||
{
|
||||
"rules": {
|
||||
"javascript-concat-html": 313,
|
||||
"javascript-escape": 7,
|
||||
"javascript-interpolate": 71,
|
||||
"javascript-jquery-append": 120,
|
||||
"javascript-jquery-html": 313,
|
||||
"javascript-jquery-insert-into-target": 26,
|
||||
"javascript-jquery-insertion": 30,
|
||||
"javascript-jquery-prepend": 12,
|
||||
"mako-html-entities": 0,
|
||||
"mako-invalid-html-filter": 33,
|
||||
"mako-invalid-js-filter": 249,
|
||||
"mako-js-html-string": 0,
|
||||
"mako-js-missing-quotes": 0,
|
||||
"mako-missing-default": 248,
|
||||
"mako-multiple-page-tags": 0,
|
||||
"mako-unknown-context": 0,
|
||||
"mako-unparseable-expression": 0,
|
||||
"mako-unwanted-html-filter": 0,
|
||||
"python-close-before-format": 0,
|
||||
"python-concat-html": 28,
|
||||
"python-custom-escape": 13,
|
||||
"python-deprecated-display-name": 53,
|
||||
"python-interpolate-html": 68,
|
||||
"python-parse-error": 0,
|
||||
"python-requires-html-or-text": 0,
|
||||
"python-wrap-html": 289,
|
||||
"underscore-not-escaped": 709
|
||||
},
|
||||
"total": 2565
|
||||
}'
|
||||
export SAFELINT_THRESHOLDS=${SAFELINT_THRESHOLDS//[[:space:]]/}
|
||||
Reference in New Issue
Block a user