Skip to content

Commit edbf64f

Browse files
Fix workflow parse failure from an empty expression in a comment
The nightly-probe comment used a literal \${{ }} to refer to GitHub expression expansion. GitHub substitutes expressions in run blocks before the shell sees them, so the empty expression made the whole workflow unparseable ("An expression was expected") and broke every dispatch mode, including source builds. Reworded; comment-only change, no behavioral difference. Introduced by the probe hardening commit 50b6a8e, which therefore never ran in Actions. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
1 parent 0b6db7f commit edbf64f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎.github/workflows/benchmark.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1523,7 +1523,7 @@ jobs:
15231523
# step output. Restrict it to a shell-safe charset at capture time so
15241524
# no shell metacharacter can ever ride along, and anchor-validate
15251525
# before use. Consumers receive it via env:, never via textual
1526-
# ${{ }} expansion into a generated script.
1526+
# GitHub expression expansion into a generated script.
15271527
filename=$(printf '%s' "$listing" | grep -oE 'OpenMS-[-A-Za-z0-9._+~]+-Debian-Linux-x86_64\.deb' | head -1)
15281528
if [[ -z "$filename" ]] || ! [[ "$filename" =~ ^OpenMS-[-A-Za-z0-9._+~]+-Debian-Linux-x86_64\.deb$ ]]; then
15291529
echo "ERROR: no x86_64 Debian .deb with the expected shell-safe filename in $base/ (see archive_listing.html in the probe artifact)" >&2

0 commit comments

Comments
 (0)