Skip to content

Conversation

@jacobtomlinson
Copy link
Member

@jacobtomlinson jacobtomlinson commented Dec 5, 2025

Currently PRs from forks do not trigger CI. This PR adds those triggers.

  • Update CONTRIBUTING.md with instructions for maintainers on running CI for forks
  • Added a workflow that comments on PRs to explain to users why CI isn't running, and to point maintainers to the docs
  • Add comments to workflow files to explain why we aren't triggering on PRs

@jacobtomlinson
Copy link
Member Author

Ah I see this is a limitation of NVIDIA hosted GitHub Actions runners. The solution is for a maintainer here to pull a PR locally once they confirm is is benign and push it back up to a branch that matches the pattern pull-request/[0-9]+. When the CI for that branch completes it will mark the PR with the results due to the commit SHAs matching.

I'll revert the changes in this PR and add some docs on this process instead.

@brycelelbach brycelelbach merged commit bbe41e3 into NVIDIA:main Dec 5, 2025
2 checks passed
@jacobtomlinson jacobtomlinson deleted the ci/gha-pr-trigger branch December 8, 2025 16:00
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