From d1aa75e3f2795d84ce3f343d36eac6157b4e5076 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Mon, 18 Oct 2021 14:50:11 -0400 Subject: [PATCH] build: run xss-commit-linter verbosely --- .github/workflows/quality-checks.yml | 2 +- pavelib/quality.py | 2 +- scripts/xss-commit-linter.sh | 5 +---- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/.github/workflows/quality-checks.yml b/.github/workflows/quality-checks.yml index 5ee54e43a7..97f28b85de 100644 --- a/.github/workflows/quality-checks.yml +++ b/.github/workflows/quality-checks.yml @@ -23,7 +23,7 @@ jobs: with: fetch-depth: 2 - - name: Fetch master for comparison + - name: Fetch base branch for comparison run: git fetch --depth=1 origin ${{ github.base_ref }} - name: Install Required System Packages diff --git a/pavelib/quality.py b/pavelib/quality.py index ccd2f124e2..c835bdf56e 100644 --- a/pavelib/quality.py +++ b/pavelib/quality.py @@ -518,7 +518,7 @@ def run_xsscommitlint(): _prepare_report_dir(xsscommitlint_report_dir) sh( - "{repo_root}/scripts/{xsscommitlint_script} | tee {xsscommitlint_report}".format( + "{repo_root}/scripts/{xsscommitlint_script} -v | tee {xsscommitlint_report}".format( repo_root=Env.REPO_ROOT, xsscommitlint_script=xsscommitlint_script, xsscommitlint_report=xsscommitlint_report, diff --git a/scripts/xss-commit-linter.sh b/scripts/xss-commit-linter.sh index bbf7cc3899..d6d144497c 100755 --- a/scripts/xss-commit-linter.sh +++ b/scripts/xss-commit-linter.sh @@ -48,7 +48,7 @@ for i in "$@"; do shift # past argument=value ;; -v|--verbose) - VERBOSE=true + show_verbose ;; -h|--help|*) # help or unknown option @@ -85,9 +85,6 @@ if [ "$diff_files" = "" ]; then echo "" echo "No files linted." else - if [ ${VERBOSE} ] ; then - show_verbose - fi for f in $diff_files; do echo "" echo "Linting $f:"