Skip to content

ci: skip workflows on forks to prevent duplicate check runs#36

Merged
dims merged 1 commit intoNVIDIA:mainfrom
dims:ci/skip-workflows-on-forks
Feb 3, 2026
Merged

ci: skip workflows on forks to prevent duplicate check runs#36
dims merged 1 commit intoNVIDIA:mainfrom
dims:ci/skip-workflows-on-forks

Conversation

@dims
Copy link
Collaborator

@dims dims commented Feb 3, 2026

Summary

Add repository check to only run workflows in NVIDIA/eidos, preventing forks from creating competing check runs.

Motivation / Context

When a fork has Actions enabled, it creates its own check runs (skipped due to branch filters) that compete with the main repo's check runs. GitHub shows the fork's skipped checks on the PR instead of the main repo's actual successful results.

This is why PR checks weren't showing correctly despite workflows running successfully on pull-request/N branches.

Fixes: PR checks not showing on fork PRs
Related: #33

Type of Change

  • Build/CI/tooling

Component(s) Affected

  • Other: .github/workflows/on-push.yaml

Implementation Notes

Added github.repository == 'NVIDIA/eidos' check to each job's if condition. This ensures:

  • Workflows only run in the main repo
  • Forks don't create competing check runs
  • Main repo's check runs are the only ones associated with PR commits

Testing

make lint  # YAML validation passes

Risk Assessment

  • Low — Isolated change, well-tested, easy to revert

Rollout notes: N/A - CI-only change

Checklist

  • Tests pass locally (make test with -race)
  • Linter passes (make lint)
  • I did not skip/disable tests to make CI green
  • Changes follow existing patterns in the codebase
  • Commits are signed off (git commit -s) — DCO info

When a fork has Actions enabled, it creates its own check runs that
compete with the main repo's check runs. The PR shows the fork's
skipped checks instead of the main repo's actual results.

Add repository check to only run workflows in NVIDIA/eidos, preventing
forks from creating competing check runs.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Davanum Srinivas <dsrinivas@nvidia.com>
@dims dims requested a review from a team as a code owner February 3, 2026 16:33
@dims dims merged commit 11e001a into NVIDIA:main Feb 3, 2026
3 checks passed
@dims dims mentioned this pull request Feb 3, 2026
2 tasks
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