Skip to content

ci(t3x): add fork-runnable PR/main verify gate - #24

Merged
radroid merged 1 commit into
mainfrom
t3x/fork-ci-pr-gate
Jul 27, 2026
Merged

ci(t3x): add fork-runnable PR/main verify gate#24
radroid merged 1 commit into
mainfrom
t3x/fork-ci-pr-gate

Conversation

@radroid

@radroid radroid commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Why

Upstream's ci.yml runs on blacksmith-* runners the fork can't access, so fork PRs currently get no checks — the "no independent CI gate" gap flagged in the review of the last sync PR (#22). Until now, "green" on a sync was only self-attested by the resolve job.

What

New fork-owned workflow .github/workflows/t3x-ci.ymlvp run typecheck / lint / test on ubuntu-latest:

  • pull_request → gates every feature PR (like this one).
  • push to main → post-sync verification that landed main is green.
  • push to t3x/sync-** → independent gate for sync branches (a conflicting sync PR can't build a merge ref, so pull_request never fires on it).

All three checks run each time (if: ${{ !cancelled() }}) so one run reports every failure. Minimal contents: read permissions, no secrets. Also listed t3x-sync-resolve.yml + t3x-ci.yml in SEAMS.md's fork-owned files.

Self-test

This PR adds the workflow, so the t3x fork CI check on this very PR is the proof it works — merging once it's green.

🤖 Generated with Claude Code

Upstream's ci.yml runs on blacksmith-* runners the fork can't access, so fork
PRs currently get no checks and "green" was only ever self-attested by the
resolve job. Add t3x-ci.yml: typecheck/lint/test on ubuntu-latest, triggered on
pull_request (gates feature PRs), push to main (post-sync verification), and
push to t3x/sync-** (independent gate for conflicting sync PRs, which
pull_request can't build a merge ref for). All three checks run each time so a
single run reports every failure.

Also list t3x-sync-resolve.yml + t3x-ci.yml in SEAMS.md's fork-owned files.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 07c48d8d-ee27-4fce-b6b6-98ad0c180b8c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch t3x/fork-ci-pr-gate

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@radroid
radroid merged commit b7cec63 into main Jul 27, 2026
2 checks passed
@radroid
radroid deleted the t3x/fork-ci-pr-gate branch July 27, 2026 13:41
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