Skip to content

Commit 5877864

Browse files
committed
ci: consolidate required checks into single ci.yaml with gate job
Replace 7 individual required-check workflows (unit-tests, mypy, 5 E2E tests) with a single ci.yaml following the astral-sh/ruff gate pattern: - determine-changes job uses native git diff (no third-party deps) - Each test job skipped at job level when paths don't match - Single required-checks-passed gate job accepts success + skipped - E2E security preserved: environment gating, author allowlists This fixes the long-standing issue where workflow-level path filters leave required checks "Pending" on PRs that don't touch code paths, blocking merge without admin override. Estimated savings: ~$1.05/skipped PR ($0.64 unit-tests + $0.01 type-check + $0.40 E2E), ~$50-100/yr in compute, plus eliminating all admin-merge workarounds.
1 parent cf59523 commit 5877864

8 files changed

Lines changed: 477 additions & 549 deletions

.github/workflows/ci.yaml

Lines changed: 477 additions & 0 deletions
Large diffs are not rendered by default.

.github/workflows/e2e-bubblesort-pytest-nogit.yaml

Lines changed: 0 additions & 92 deletions
This file was deleted.

.github/workflows/e2e-bubblesort-unittest.yaml

Lines changed: 0 additions & 76 deletions
This file was deleted.

.github/workflows/e2e-futurehouse-structure.yaml

Lines changed: 0 additions & 76 deletions
This file was deleted.

.github/workflows/e2e-topological-sort.yaml

Lines changed: 0 additions & 101 deletions
This file was deleted.

0 commit comments

Comments
 (0)