Skip to content

ci: consolidate PR checks into single Validate Code gate#175

Merged
rhuanbarreto merged 2 commits into
mainfrom
ci/consolidate-pr-checks
Mar 25, 2026
Merged

ci: consolidate PR checks into single Validate Code gate#175
rhuanbarreto merged 2 commits into
mainfrom
ci/consolidate-pr-checks

Conversation

@rhuanbarreto

Copy link
Copy Markdown
Contributor

Summary

  • Merge smoke-test.yml into code-pull-request.yml — all PR checks now live in one workflow
  • Add "Validate Code" gate job — a lightweight job that needs: [validate, smoke-windows, smoke-linux] and fails if any dependency fails
  • Single required status check — set "Validate Code" as the only required check in branch protection. If validate, smoke tests, llms-full.txt check, or lockfile check fails, the PR is blocked.
  • Delete deploy-docs.yml — docs are deployed via Cloudflare Pages

How it works

┌─────────────────────┐  ┌──────────────────────┐  ┌─────────────────────┐
│  Lint, Test & Check  │  │ Smoke Test (Windows)  │  │ Smoke Test (Linux)  │
│  (ubuntu-latest)     │  │ (windows-latest)      │  │ (ubuntu-latest)     │
└──────────┬──────────┘  └──────────┬───────────┘  └──────────┬──────────┘
           │                        │                          │
           └────────────┬───────────┴──────────────────────────┘
                        ▼
              ┌──────────────────┐
              │  Validate Code   │  ← single required check
              │  (gate job)      │
              └──────────────────┘

The update-llms.yaml and update-lock.yaml workflows remain separate — they auto-commit fixes to the PR branch using a GitHub App token. If they fail, the validate job catches the stale state (llms-full.txt check, frozen lockfile).

Test plan

  • bun run validate passes locally
  • Verify "Validate Code" appears as a status check on the PR
  • Update branch protection to require only "Validate Code"

Merge smoke-test.yml into code-pull-request.yml and add a gate job
("Validate Code") that depends on all other jobs. This provides a
single required status check for branch protection that blocks the PR
if any job fails — validate, Windows smoke test, or Linux smoke test.

Also removes deploy-docs.yml (docs deployed via Cloudflare Pages).
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Mar 25, 2026

Copy link
Copy Markdown

Deploying archgate-cli with  Cloudflare Pages  Cloudflare Pages

Latest commit: ff68b16
Status: ✅  Deploy successful!
Preview URL: https://45aee566.archgate-cli.pages.dev
Branch Preview URL: https://ci-consolidate-pr-checks.archgate-cli.pages.dev

View logs

Extract Windows and Linux smoke tests into separate workflow files
called via workflow_call. The orchestrator (code-pull-request.yml)
stays lean — validate job + two smoke test calls + gate job.
@rhuanbarreto rhuanbarreto merged commit cd061c9 into main Mar 25, 2026
9 checks passed
@rhuanbarreto rhuanbarreto deleted the ci/consolidate-pr-checks branch March 25, 2026 02:28
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