Skip to content

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

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#425
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 marianogappa 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

Updates the PR title validation GitHub Actions workflow to run on the pull_request event rather than pull_request_target, reducing exposure to untrusted fork code while still validating titles via the PR event payload.

Changes:

  • Switch 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 26eb2ce into main Mar 25, 2026
7 checks passed
@kodiakhq kodiakhq bot deleted the chore/replace-pull-request-target branch March 25, 2026 16:11
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