Skip to content

chore: Replace pull_request_target with pull_request in PR title validation#377

Merged
kodiakhq[bot] merged 2 commits intomainfrom
chore/replace-pull-request-target
Mar 25, 2026
Merged

chore: Replace pull_request_target with pull_request in PR title validation#377
kodiakhq[bot] merged 2 commits intomainfrom
chore/replace-pull-request-target

Conversation

@erezrokah
Copy link
Member

Replace pull_request_target with pull_request to reduce attack surface. The semantic PR title action only reads the event payload, so elevated permissions are not needed.

@erezrokah erezrokah requested a review from a team as a code owner March 25, 2026 14:04
@erezrokah erezrokah requested review from Copilot and murarustefaan and removed request for a team March 25, 2026 14:04
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the PR title validation GitHub Actions workflow to use the pull_request event instead of pull_request_target, reducing the workflow’s attack surface while keeping the same functionality (the action only needs PR metadata).

Changes:

  • Switched the workflow trigger from pull_request_target to pull_request for PR title validation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@erezrokah erezrokah added the automerge Add to automerge PRs once requirements are met label Mar 25, 2026
@kodiakhq kodiakhq bot merged commit 146bb00 into main Mar 25, 2026
8 checks passed
@kodiakhq kodiakhq bot deleted the chore/replace-pull-request-target branch March 25, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Add to automerge PRs once requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants