Skip to content

Commit 7fc8753

Browse files
authored
fix(release): use case-insensitive label matching in check_release (#158)
## What/Why The old GitHub Actions `contains()` expression was case-insensitive, but the bash replacement introduced in #156 compared labels case-sensitively. Lowercase labels with `${PR_LABELS,,}` to preserve the original semantics. ## Proof it works actionlint and shellcheck both pass with no errors. ## Risk + AI role Low -- one-liner fix. AI-assisted (Claude Opus 4.6). ## Review focus Confirm `${PR_LABELS,,}` (bash 4+ lowercase expansion) works on the ubuntu-latest runner. Signed-off-by: jmeridth <jmeridth@gmail.com>
1 parent dc84e76 commit 7fc8753

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/release.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ jobs:
106106
elif [ "$PR_MERGED" != "true" ]; then
107107
echo "PR not merged: skipping release"
108108
else
109-
IFS=',' read -ra labels <<< "$PR_LABELS"
109+
IFS=',' read -ra labels <<< "${PR_LABELS,,}"
110110
if [ "$RELEASE_ONLY" = "true" ]; then
111111
for label in "${labels[@]}"; do
112112
if [ "$label" = "release" ]; then

0 commit comments

Comments
 (0)