Skip to content

Commit 3604cbf

Browse files
committed
Fix CI for check-role-prefix script
We're skipping Single quotes in sed as we want literal $ in the sed pattern, not shell expansion. Added this file to irrelevant-files.
1 parent 5bf2fce commit 3604cbf

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

scripts/check-role-prefix.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ MSG=$(head -n 1 "$TMP_MSG_FILE")
2424
ROLE_COUNT=$(echo "$CHANGED_ROLES" | tr '|' '\n' | wc -l)
2525

2626
if [ "$ROLE_COUNT" -eq 1 ]; then
27-
# shellcheck disable=SC1087
27+
# shellcheck disable=SC2016
2828
ESCAPED_ROLE=$(printf '%s\n' "$CHANGED_ROLES" | sed 's/[]\.*^$()+?{|]/\\&/g')
29-
PATTERN="^[[(]$ESCAPED_ROLE[])]"
29+
PATTERN="^[[(]${ESCAPED_ROLE}[])]"
3030
else
3131
PATTERN="^[[(](multiple)[])]"
3232
fi

zuul.d/base.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353
- .pre-commit-config.yaml
5454
- .readthedocs.yaml
5555
- .spellcheck.yml
56+
- ^scripts/check-role-prefix.sh$
5657
- roles/dlrn_report
5758
- roles/dlrn_promote
5859
- roles/devscripts

0 commit comments

Comments
 (0)