Skip to content

Add fork-safe CI workflow for external contributors#173

Merged
raymondjacobson merged 2 commits intomainfrom
fork-ci-support
Mar 31, 2026
Merged

Add fork-safe CI workflow for external contributors#173
raymondjacobson merged 2 commits intomainfrom
fork-ci-support

Conversation

@raymondjacobson
Copy link
Copy Markdown
Contributor

@raymondjacobson raymondjacobson commented Mar 31, 2026

Summary

  • Adds a separate ci-fork.yml workflow that runs on pull_request (no secrets needed)
  • Runs Go unit tests directly (go test) without Docker builds for fork PRs
  • Same-repo PRs continue using the existing ci.yml with full Docker build/test/push
  • No pull_request_target, no secrets exposure — fork PRs only get read-only access

Test plan

  • Same-repo PR still triggers ci.yml normally
  • Fork PR triggers ci-fork.yml and runs unit tests
  • Fork PR does NOT trigger ci.yml Docker build/push

🤖 Generated with Claude Code

Adds a separate ci-fork.yml that triggers on pull_request (no secrets
needed). Runs Go unit tests directly without Docker for fork PRs.
Same-repo PRs continue using the existing ci.yml with full Docker
build/test/push pipeline.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@raymondjacobson raymondjacobson changed the title Allow CI to run on fork PRs with manual approval Add fork-safe CI workflow for external contributors Mar 31, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@raymondjacobson raymondjacobson merged commit 0d4e1ed into main Mar 31, 2026
4 checks passed
@raymondjacobson raymondjacobson deleted the fork-ci-support branch March 31, 2026 01:25
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.

1 participant