Skip to content

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

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

chore: Replace pull_request_target with pull_request in PR title validation#361
kodiakhq[bot] merged 1 commit intomainfrom
chore/replace-pull-request-target

Conversation

@erezrokah
Copy link
Copy Markdown
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 disq and removed request for a team March 25, 2026 14:04
Copy link
Copy Markdown

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 reduces GitHub Actions attack surface by switching the PR title validation workflow trigger from pull_request_target to pull_request, since the semantic title check only needs PR event data and read access.

Changes:

  • Replace the workflow trigger pull_request_target with 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 32d3554 into main Mar 25, 2026
14 checks passed
@kodiakhq kodiakhq bot deleted the chore/replace-pull-request-target branch March 25, 2026 16:17
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