Skip to content

[ci]: add fork-safe lint and CPU PR gate - #6461

Merged
yuxuan-lou merged 1 commit into
mainfrom
ci/fork-cpu-ci
Oct 10, 2026
Merged

yuxuan-lou merged 1 commit into
mainfrom
ci/fork-cpu-ci

Conversation

@richardoo-707

@richardoo-707 richardoo-707 commented Oct 8, 2026 •

Copy link
Copy Markdown
Collaborator

🚨 Issue number

N/A — CI infrastructure change.

📝 What does this PR do?

  • Adds a fork-safe pull request gate on GitHub-hosted Ubuntu runners.
  • Separates repository lint/formatting from CPU-only unit tests.
  • Uses read-only permissions, does not expose repository secrets, and disables persisted checkout credentials.
  • Restricts existing private/self-hosted PR jobs that execute unmerged code to same-repository branches.
  • Adds an automated runner-policy validator and unit tests.
  • Makes the shared pytest setup compatible with the CPU accelerator.

This change builds on the existing runner foundation from E1, reuses the empty-example false-green coverage from E3, and depends on the CPU test subset completed in E4. It does not replace or conflict with those projects.

✅ Test plan

  • Changed-file pre-commit checks
  • Runner-policy unit tests: 5 passed
  • GitHub Actions Lint and formatting
  • GitHub Actions CPU unit tests: 18 passed with PyTorch 2.5.1+cpu and no visible GPU
  • GitHub Actions two-GPU smoke test
  • Workflow YAML and actionlint validation

Two legacy self-hosted change-detection jobs may remain queued when no matching {self-hosted, ubuntu-latest} runner is available; they are independent of the fork-safe gate added here.

💥 Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added thorough tests.
  • I have documented the new workflow and runner-policy requirements.

@richardoo-707
richardoo-707 requested a review from a team as a code owner October 8, 2026 10:11
@richardoo-707 richardoo-707 changed the title ci: add fork-safe CPU pull request gate [ci]: add fork-safe lint and CPU PR gate Oct 8, 2026
@yuxuan-lou
yuxuan-lou merged commit df01070 into main Oct 10, 2026
8 of 11 checks passed
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