Skip to content

Clarify explicit PR targets rejected by triggering-only push configuration - #65065

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-push-to-pull-request-error
Closed

pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-push-to-pull-request-error

Conversation

Copilot AI commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

On scheduled runs, push_to_pull_request_branch could report that target "triggering" lacked PR context even when the caller supplied a PR number. The error did not explain why that number was ignored.

  • Diagnostic: Name a supplied numeric pull_request_number and explain that per-call PR targeting requires target: "*".
  • Target boundary: Keep target: "triggering" context-bound; a per-call number does not override the workflow configuration.
  • Regression coverage: Cover the scheduled-run case with an explicit PR number.

@pelikhan pelikhan closed this Oct 2, 2026
Copilot AI changed the title [WIP] Fix explicit target consumption in push_to_pull_request_branch Clarify explicit PR targets rejected by triggering-only push configuration Oct 2, 2026
Copilot AI requested a review from pelikhan October 2, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

push_to_pull_request_branch: explicit target not consumed; error names the "triggering" default instead

2 participants