Skip to content

Commit 7e6c199

Browse files
fix(ci): guard changelog printf bullet formats
Use printf -- for post-merge changelog bullet lines so runner shells do not treat leading hyphens in format strings as options. Record the failed PR #13 workflow run and local reproduction proof in the US-025 story evidence.
1 parent bfef94a commit 7e6c199

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

.github/workflows/post-merge-maintenance.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,16 +75,16 @@ jobs:
7575
entry="$(mktemp)"
7676
{
7777
printf '## %s - PR #%s\n\n' "$date_utc" "$PR_NUMBER"
78-
printf '- %s (@%s)\n' "$pr_title" "$pr_author"
78+
printf -- '- %s (@%s)\n' "$pr_title" "$pr_author"
7979
if [ -n "$merge_sha" ]; then
80-
printf '- Merge commit: `%s`\n' "$merge_sha"
80+
printf -- '- Merge commit: `%s`\n' "$merge_sha"
8181
fi
8282
if [ "$cli_changed" = true ]; then
83-
printf '- Harness CLI release: `%s`\n' "$release_tag"
83+
printf -- '- Harness CLI release: `%s`\n' "$release_tag"
8484
else
85-
printf '- Harness CLI release: not required\n'
85+
printf -- '- Harness CLI release: not required\n'
8686
fi
87-
printf '- Changed files:\n'
87+
printf -- '- Changed files:\n'
8888
sed 's/^/ - `/' <<<"$changed_files" | sed 's/$/`/'
8989
printf '\n'
9090
} > "$entry"

docs/stories/US-025-post-merge-cli-release-and-changelog.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,3 +64,8 @@ conditional CLI release preparation after merged pull requests.
6464
- `cargo clippy --workspace -- -D warnings`: passed.
6565
- `bash -n scripts/install-harness.sh && bash -n scripts/build-harness-cli-release.sh`: passed.
6666
- `actionlint`: not installed locally, so GitHub-specific workflow linting was not run.
67+
- 2026-06-09 follow-up: GitHub run `27180707313` failed in
68+
`Update maintenance files` because changelog bullet `printf` formats began
69+
with `-` and were parsed as options on the runner. The workflow now uses
70+
`printf --` for all bullet formats. Local reproduction of the PR #13
71+
changelog entry passed.

0 commit comments

Comments
 (0)