Skip to content

Switch Claude Code review to pull_request_target#1357

Closed
syed-ahsan-ishtiaque wants to merge 1 commit intomainfrom
switch-to-pull-request-target
Closed

Switch Claude Code review to pull_request_target#1357
syed-ahsan-ishtiaque wants to merge 1 commit intomainfrom
switch-to-pull-request-target

Conversation

@syed-ahsan-ishtiaque
Copy link
Copy Markdown
Contributor

Summary

  • Switches the Claude Code review workflow from pull_request to pull_request_target
  • Fixes review failures on fork PRs, dependabot PRs, and first-time contributor PRs which don't have access to secrets under pull_request
  • pull_request_target runs the workflow from the base branch (secure) and provides secrets access for all PR types

Test plan

  • Verified pull_request_target works on Java repo (successful run)
  • Verify fork/dependabot PRs now get Claude reviews

@syed-ahsan-ishtiaque syed-ahsan-ishtiaque added the skip changelog doesn't need a CHANGELOG entry label Apr 15, 2026
@syed-ahsan-ishtiaque
Copy link
Copy Markdown
Contributor Author

Closing: main already has pull_request_target (fixed separately).

@syed-ahsan-ishtiaque syed-ahsan-ishtiaque deleted the switch-to-pull-request-target branch April 15, 2026 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog doesn't need a CHANGELOG entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant