Skip to content

workflows: use pull_request instead of pull_request_target - #4474

Merged
rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:convert_pull_request_target_workflows
Nov 11, 2026
Merged

rickeylev merged 1 commit into
bazel-contrib:mainfrom
rickeylev:convert_pull_request_target_workflows

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

pull_request_target runs in the privileged base-branch context,
which is unnecessary and risky for checks that only read PR labels,
descriptions, and diff filenames.

Switch the PR metadata checks workflow trigger to pull_request so
checks run in the unprivileged pull request context.

`pull_request_target` runs in the privileged base-branch context,
which is unnecessary and risky for checks that only read PR labels,
descriptions, and diff filenames.

Switch the PR metadata checks workflow trigger to `pull_request` so
checks run in the unprivileged pull request context.
@rickeylev
rickeylev requested a review from aignas as a code owner November 11, 2026 04:49
@rickeylev

Copy link
Copy Markdown
Collaborator Author

This pull request was prepared and submitted with AI agent assistance. Review comments are monitored by the human author, who reviews all feedback and either responds directly or reviews and approves any agent-assisted updates before they are posted.

This is an automatically generated agent comment.

@rickeylev
rickeylev merged commit 71573b1 into bazel-contrib:main Nov 11, 2026
17 checks passed
@rickeylev
rickeylev deleted the convert_pull_request_target_workflows branch November 11, 2026 06:04
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.

1 participant