From 4be2a74afa35d59348696f299e1e10f7cd2a2f5f Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 11 Oct 2026 18:55:06 +0000 Subject: [PATCH] CI: drop redundant build in nightly test job The macOS/Windows `test` legs ran `cargo build --workspace` before `cargo test`, which builds every target the tests need anyway (the CLI binary included). The extra step cost about 2 minutes per leg. The Linux `coverage` job already runs the same test selection with no pre-build, so the step bought nothing. The `test` job only runs on full scope (nightly / dispatch), so PR and merge-queue runner usage is unchanged. Refs #1173 Assisted-by: Claude Code:claude-opus-5-5 --- .github/workflows/ci.yml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c1e4c5509..3bd152af9 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -372,9 +372,10 @@ jobs: shared-key: dev-${{ matrix.os }} save-if: ${{ github.ref == 'refs/heads/main' }} - - name: Build - run: cargo build --workspace - + # No separate `cargo build --workspace` step: `Run tests` (or the + # warm step below) builds every workspace target the tests need, + # the CLI binary included. The extra build cost ~2 min per leg + # (#1173); `coverage` runs the same selection on Linux without one. - name: Warm the test cache after merge-queue validation if: needs.clippy.outputs.reuse == 'true' run: cargo test --locked --workspace --no-run