Skip to content

[codex] add required CI gate#519

Merged
blove merged 2 commits into
mainfrom
codex/fix-ci-required-gate
May 21, 2026
Merged

[codex] add required CI gate#519
blove merged 2 commits into
mainfrom
codex/fix-ci-required-gate

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 21, 2026

Summary

  • Add a stable CI — required pull-request gate that waits for every scoped CI job and fails if any selected job fails, is cancelled, or is skipped unexpectedly.
  • Fix the cockpit e2e summary job dependency so it can read ci-scope outputs while preserving the existing Cockpit — e2e check name.
  • Point deploy prerequisites at the cockpit e2e summary instead of the raw matrix job.

Root Cause

Branch protection only required Vercel – threadplane, so GitHub auto-merge could merge a PR while GitHub Actions jobs were still running or while non-required CI jobs were failing. GitHub auto-merge waits for required status checks, not every check in a workflow.

Branch Protection

main branch protection has already been updated to require:

  • Vercel – threadplane
  • CI — required

Strict required status checks are enabled.

Test Plan

  • node --test scripts/ci-workflow.spec.mjs
  • node --test scripts/ci-scope.spec.mjs
  • YAML parse of .github/workflows/ci.yml
  • git diff --check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment May 21, 2026 8:09pm

Request Review

@blove blove merged commit 99e2d48 into main May 21, 2026
2 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.

1 participant