Skip to content

Conversation

@ashwin-ant
Copy link
Collaborator

Summary

  • Add if conditions to skip e2e test jobs (test-e2e, test-e2e-docker, test-examples) when PRs come from forks
  • Add if condition to skip claude-review job for fork PRs
  • Prevents CI failures for external contributors since fork PRs don't have access to repo secrets like ANTHROPIC_API_KEY

Test plan

  • Verify that unit tests still run for fork PRs (the test job has no condition)
  • Verify that e2e jobs are skipped for fork PRs
  • Verify that e2e jobs still run for PRs from the main repo
  • Verify that push to main still triggers all jobs

🤖 Generated with Claude Code

Add conditional checks to skip CI jobs that require ANTHROPIC_API_KEY
when PRs come from forks, since fork PRs don't have access to repo
secrets. This prevents unnecessary CI failures for external contributors.

Affected jobs:
- test-e2e
- test-e2e-docker
- test-examples
- claude-review

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@ashwin-ant ashwin-ant requested a review from a team January 5, 2026 10:46
@ashwin-ant ashwin-ant merged commit 23c9df3 into main Jan 5, 2026
17 of 29 checks passed
@ashwin-ant ashwin-ant deleted the ashwin/skip-ci-jobs-on-forks branch January 5, 2026 16:58
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.

4 participants