Commit 0735a1c
committed
workflows: Fix broken protected-files override expression
The && ternary idiom (cond && 'allowed' || 'request_review') broke
drafter.lock.yml and fix.lock.yml outright: gh-aw's compiler
JSON-encodes this policy string with Go's default HTML-escaping,
turning the literal && into \u0026\u0026 inside the heredoc-embedded
config blob. GitHub Actions' expression parser can't parse that,
so the whole workflow file was rejected as invalid on push -- see
https://github.com/cgwalters/gh-aw-fullsend-mini/actions/runs/30547055376
(zero jobs even attempted; both drafter.md and fix.md were
effectively disabled since the merge of #20).
Switch to GitHub Actions' case() function, which expresses the same
if/else without &, <, or > -- the only characters Go's json.Marshal
HTML-escapes by default -- so it survives gh-aw's config serialization
intact.
Assisted-by: https://github.com/cgwalters/cgwalters#llms1 parent 50f91dd commit 0735a1c
4 files changed
Lines changed: 22 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
42 | 48 | | |
43 | 49 | | |
44 | 50 | | |
| |||
0 commit comments