Skip to content

Commit d39d9fa

Browse files
committed
Move type+audit checks into matrix rows, remove gate job
1 parent 0513ce2 commit d39d9fa

1 file changed

Lines changed: 7 additions & 19 deletions

File tree

.github/workflows/test.yml

Lines changed: 7 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -22,32 +22,13 @@ on:
2222
paths-ignore: ['*.md', '**/docs/**']
2323

2424
jobs:
25-
# ── Quick-check gate — catch trivial failures before the full matrix ──
26-
quick-check:
27-
runs-on: ubuntu-latest
28-
steps:
29-
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
30-
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
31-
with:
32-
node-version: "22"
33-
cache: "npm"
34-
- run: npm ci
35-
36-
# Fast pre-flight checks — type errors and security issues before matrix
37-
- name: Type check
38-
run: npx tsc --noEmit
39-
40-
- name: Security audit
41-
run: npm audit --audit-level=moderate
42-
4325
# ── Cross-platform test matrix ──────────────────────────────────────
4426
# Node 22 (minimum) is tested only on Linux — the primary platform and the only one
4527
# guaranteed to have the oldest toolchain. macOS and Windows test Node 24 (latest)
4628
# to catch regressions in the newest runtime. This asymmetry is intentional: it
4729
# balances CI cost with meaningful coverage while ensuring the minimum version works
4830
# correctly on the platform most likely to encounter toolchain edge cases.
4931
test:
50-
needs: quick-check
5132
runs-on: ${{ matrix.os }}
5233
strategy:
5334
fail-fast: false # report every combination, don't cancel
@@ -71,6 +52,13 @@ jobs:
7152

7253
- run: npm ci
7354

55+
# Uniform pre-flight checks — type errors and security issues on every platform
56+
- name: Type check
57+
run: npx tsc --noEmit
58+
59+
- name: Security audit
60+
run: npm audit --audit-level=moderate
61+
7462
# Unit suite (unit tests + snapshot tests + property-based tests)
7563
- name: Unit tests
7664
run: npm test

0 commit comments

Comments
 (0)