Skip to content

.sync: Use pull_request_target for Rust version check workflow#136

Merged
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:ver_check_use_pr_target_trigger
May 7, 2026
Merged

.sync: Use pull_request_target for Rust version check workflow#136
makubacki merged 1 commit into
OpenDevicePartnership:mainfrom
makubacki:ver_check_use_pr_target_trigger

Conversation

@makubacki
Copy link
Copy Markdown
Collaborator

@makubacki makubacki commented May 6, 2026

This workflow needs to use secrets for PRs from public forks, which is not available with the pull_request trigger currently used.

pull_request_target is used instead since it allows access to secrets and the workflow is limited to simple toml parsing and validation not executing any code from the PR.


Tested on a pull request to a PR from a public fork here: makubacki/patina#13

This workflow needs to use secrets for PRs from public forks, which
is not available with the pull_request trigger currently used.

pull_request_target is used instead since it allows access to secrets
and the workflow is limited to simple toml parsing and validation not
executing any code from the PR.

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
@makubacki makubacki requested review from Javagedes, apop5 and vineelko May 6, 2026 22:43
@makubacki makubacki self-assigned this May 6, 2026
@makubacki makubacki merged commit 8515061 into OpenDevicePartnership:main May 7, 2026
2 checks passed
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.

4 participants