From 654f6839ebe33d64710b5e7f3c388539e84f8e35 Mon Sep 17 00:00:00 2001
From: Harmen Stoppels <harmenstoppels@gmail.com>
Date: Thu, 18 Nov 2021 13:00:39 +0100
Subject: ci: run style unit tests only if we target develop (#27472)
Some tests assume the base branch is develop, but this branch may not
have been checked out.
---
share/spack/qa/run-style-tests | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
(limited to 'share')
diff --git a/share/spack/qa/run-style-tests b/share/spack/qa/run-style-tests
index 72c47ff055..9f0cbdb266 100755
--- a/share/spack/qa/run-style-tests
+++ b/share/spack/qa/run-style-tests
@@ -14,15 +14,15 @@
# Usage:
# run-flake8-tests
#
-. "$(dirname $0)/setup.sh"
+. "$(dirname "$0")/setup.sh"
-BASE=""
-if [ -n "$GITHUB_BASE_REF" ]; then
- BASE="--base ${GITHUB_BASE_REF}"
+args=()
+if [[ -n $GITHUB_BASE_REF ]]; then
+ args+=("--base" "${GITHUB_BASE_REF}")
fi
# verify that the code style is correct
-spack style --root-relative $BASE
+spack style --root-relative "${args[@]}"
# verify that the license headers are present
spack license verify
--
cgit v1.2.3-70-g09d2