Skip to content

ci: use pull_request_target for PR title check comments#20

Merged
dhandal09 merged 1 commit intomainfrom
ci/pr-title-check-use-pull-request-target
Mar 23, 2026
Merged

ci: use pull_request_target for PR title check comments#20
dhandal09 merged 1 commit intomainfrom
ci/pr-title-check-use-pull-request-target

Conversation

@dhandal09
Copy link
Copy Markdown
Collaborator

Summary

  • Replaces pull_request with pull_request_target so the workflow has write permissions for fork PRs without needing a GitHub App token.
  • Removes the GitHub App token generation that was added in ci: use GitHub App token for PR title check comments #17, reverting comment authorship back to github-actions[bot].
  • This is safe because the workflow only reads the PR title from the event payload and never checks out or executes PR code.

Test plan

  • Open a PR from a fork and verify the title check workflow runs and posts a comment
  • Edit a PR title and confirm the existing comment is updated (not duplicated)
  • Verify the workflow still works for non-fork PRs

Made with Cursor

Switch from pull_request to pull_request_target so the workflow
has write permissions for fork PRs without needing a GitHub App
token. This is safe because the workflow only reads the PR title
from the event payload and never checks out PR code.

Made-with: Cursor
@dhandal09 dhandal09 requested a review from a team March 23, 2026 19:09
@dhandal09 dhandal09 merged commit b5d2c88 into main Mar 23, 2026
1 check passed
@dhandal09 dhandal09 deleted the ci/pr-title-check-use-pull-request-target branch March 23, 2026 19:18
@poly-adk-bot
Copy link
Copy Markdown

poly-adk-bot bot commented Mar 23, 2026

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants