Skip to content

fix: switch pull_request_target to pull_request#16

Open
keerthi-go wants to merge 1 commit intoqualcomm-linux:mainfrom
keerthi-go:dev/fix-pull-request-target
Open

fix: switch pull_request_target to pull_request#16
keerthi-go wants to merge 1 commit intoqualcomm-linux:mainfrom
keerthi-go:dev/fix-pull-request-target

Conversation

@keerthi-go
Copy link
Copy Markdown
Contributor

  • Replace pull_request_target with pull_request to prevent fork PRs from running with elevated permissions and secret access
  • Remove conditional fork-safe checkout; use single actions/checkout@v4
  • Fix upload condition to trigger on push/schedule/workflow_dispatch only
  • Add packages: write permission required for GHCR push

- Replace pull_request_target with pull_request to prevent fork PRs
  from running with elevated permissions and secret access
- Remove conditional fork-safe checkout; use single actions/checkout@v4
- Fix upload condition to trigger on push/schedule/workflow_dispatch only
- Add packages: write permission required for GHCR push

Signed-off-by: Keerthi Gowda <kbalehal@qti.qualcomm.com>
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