Skip to content

fix(ci): quote the labeler trigger's echo so YAML keeps the whole command - #2822

Merged
milenkovicm merged 1 commit into
apache:mainfrom
andygrove:ci/fix-labeler-trigger-echo
Nov 9, 2026
Merged

milenkovicm merged 1 commit into
apache:mainfrom
andygrove:ci/fix-labeler-trigger-echo

Conversation

@andygrove

Copy link
Copy Markdown
Member

Which issue does this PR close?

No issue filed. This is a one-line CI fix for a check that fails on every pull request.

Rationale for this change

The Trigger job in dev_pr_trigger.yml (added in #2778) fails on every pull request with:

line 1: unexpected EOF while looking for matching `"'

The step is run: echo "Labels for pull request #$PR_NUMBER are applied by dev_pr.yml". Because the value is a plain YAML scalar, # starts a comment, so the runner gets only echo "Labels for pull request and the shell reports the unterminated quote. Labels still get applied, since dev_pr.yml doesn't check the trigger run's conclusion, but every PR shows a red check.

What changes are included in this PR?

The run: value is now single-quoted, so YAML keeps the whole command. I parsed the file before and after the change to confirm that the old version is cut off at # and the new one keeps the full command.

Are there any user-facing changes?

No.

…mand

An unquoted ` #` starts a YAML comment, so the run step was truncated to
`echo "Labels for pull request` and failed with an unterminated quote.

@villebro villebro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix

@NoahKusaba

Copy link
Copy Markdown
Contributor

@milenkovicm this fixes the broken ci btw

@milenkovicm
milenkovicm added this pull request to the merge queue Nov 9, 2026
Merged via the queue into apache:main with commit 10fd932 Nov 9, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants