Skip to content

pre-commit: enable pushing of auto-fixes for forks#9036

Merged
alexreinking merged 1 commit intomainfrom
alexreinking/pre-commit/allow-forks
Mar 15, 2026
Merged

pre-commit: enable pushing of auto-fixes for forks#9036
alexreinking merged 1 commit intomainfrom
alexreinking/pre-commit/allow-forks

Conversation

@alexreinking
Copy link
Copy Markdown
Member

pull_request workflows don't have access to repository secrets when launched from forks because third-party code is potentially untrusted. So we need to split into two workflows: one that just runs pre-commit, and one that grabs the diff and pushes it using our Halide CI app. The workflow_run trigger does have access to repository secrets.

This was written by Claude. Need to add the Co-authored-by tag on merge.

@alexreinking alexreinking requested a review from mcourteaux March 15, 2026 13:58
@alexreinking alexreinking merged commit 21ae307 into main Mar 15, 2026
1 check passed
@alexreinking alexreinking deleted the alexreinking/pre-commit/allow-forks branch March 15, 2026 15:12
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.

2 participants