Skip to content

Comments

fix(ci): use pull_request_target for fork pr support#3534

Merged
FabianKramm merged 1 commit intomainfrom
fix/claude-workflow-permissions
Jan 28, 2026
Merged

fix(ci): use pull_request_target for fork pr support#3534
FabianKramm merged 1 commit intomainfrom
fix/claude-workflow-permissions

Conversation

@Piotr1215
Copy link
Contributor

@Piotr1215 Piotr1215 commented Jan 28, 2026

Summary

Related

Fixes failure from e35caab

@Piotr1215 Piotr1215 force-pushed the fix/claude-workflow-permissions branch from 4b6656c to 84fceeb Compare January 28, 2026 17:21
@loft-bot
Copy link
Contributor


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


Fork PRs can't access org secrets with pull_request trigger.
Using pull_request_target runs workflow in base repo context,
allowing access to ANTHROPIC_API_KEY for Claude reviews.
@Piotr1215 Piotr1215 force-pushed the fix/claude-workflow-permissions branch from 84fceeb to 15718b9 Compare January 28, 2026 17:28
@Piotr1215 Piotr1215 changed the title fix(ci): claude workflows use explicit github token fix(ci): use pull_request_target for fork pr support Jan 28, 2026
@FabianKramm FabianKramm merged commit 4abf19a into main Jan 28, 2026
37 checks passed
@FabianKramm FabianKramm deleted the fix/claude-workflow-permissions branch January 28, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants