fix(ci): pr-review.yml parse error — literal ${...} in a comment broke the live pipeline #367

Merged
k.renner merged 1 commits from claude-bot/fix-pr-review-var-comment into master 2026-07-23 10:52:31 +00:00
+6 -3
View File
@@ -49,9 +49,12 @@
# Triggering: pull_request only — automatic on PR open + push-to-open-PR. To
# re-review a PR, push a commit (the new head sha clears the dedup marker below
# and earns a fresh review). There is deliberately no manual PIPELINE trigger:
# Woodpecker exposes `--var` values to YAML `${...}` substitution and `when`
# filters only, NOT as a step's shell environment, so a `--var PR=<n>` re-review
# would silently no-op (the step would find no PR number and exit green).
# Woodpecker exposes a --var only to YAML variable substitution and `when`
# filters, NOT to a step's shell environment, so a --var PR=<n> re-review would
# silently no-op (the step would find no PR number and exit green).
# (Note: keep literal dollar-brace out of THIS file entirely, even in comments —
# Woodpecker runs its variable substitution over the whole YAML text, comments
# included, and an unparseable one errors the pipeline before any step runs.)
when:
- event: pull_request