From 2573547753f4fc0cd2a815457a45ba06f9093eef Mon Sep 17 00:00:00 2001 From: Claude Bot Date: Thu, 23 Jul 2026 12:50:29 +0200 Subject: [PATCH] fix(ci): remove literal ${...} from pr-review.yml comment MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Woodpecker runs its ${...} variable substitution over the ENTIRE pipeline YAML including comments, so the literal dollar-brace in the triggering comment (added when documenting the dropped manual path) failed to parse — 'unable to parse variable name' — erroring every pull_request pipeline before any step ran. Reword the comment to avoid the glyph; the live homelab pipeline was down since #362 merged. Co-Authored-By: Claude Opus 4.8 (1M context) --- .woodpecker/pr-review.yml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.woodpecker/pr-review.yml b/.woodpecker/pr-review.yml index 846ba963..02847bb9 100644 --- a/.woodpecker/pr-review.yml +++ b/.woodpecker/pr-review.yml @@ -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=` 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= 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 -- 2.54.0