Skip to content

Commit e112dff

Browse files
committed
ci: add actionlint config suppressing SC2129 in workflow files
Companion to 0004: the validator continues to flag SC2129 ("Consider using { cmd1; cmd2; } >> file instead of individual redirects") at style severity in updates.yaml — the create_pr_update block has five sequential `>> "$GITHUB_OUTPUT"` redirects interleaved with conditional logic, so the linearized form reads more clearly than the bundled brace-group equivalent. Suppress repo-wide via `.github/actionlint.yaml` rather than per-block shellcheck directive: applies consistently if a future workflow grows a similar redirect chain, and keeps the workflow files themselves free of inline lint suppressions. Refs: PSEC-923
1 parent 71cacdf commit e112dff

1 file changed

Lines changed: 13 additions & 0 deletions

File tree

.github/actionlint.yaml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# Repo-wide actionlint configuration.
2+
#
3+
# Suppression rationale:
4+
# - SC2129 is a shellcheck *style* finding suggesting `{ cmd1; cmd2; } >> file`
5+
# instead of N individual `>> file` redirects. The linearized form is more
6+
# readable in GitHub Actions `run:` blocks where conditional logic is
7+
# interleaved between redirects (e.g., updates.yaml's create_pr_update
8+
# flag), so we suppress this rule repo-wide rather than refactor every
9+
# block.
10+
paths:
11+
'.github/workflows/**':
12+
ignore:
13+
- 'shellcheck reported issue.*SC2129'

0 commit comments

Comments
 (0)