Skip to content

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

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#623
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 dcelasun 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 workflow to trigger on pull_request instead of pull_request_target, reducing exposure to elevated permissions while keeping the same validation behavior.

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 b5ab8d8 into main Mar 25, 2026
10 checks passed
@kodiakhq kodiakhq bot deleted the chore/replace-pull-request-target branch March 25, 2026 16:31
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