From cbb0c52c1434afee59e1e8394772aea14ba7b742 Mon Sep 17 00:00:00 2001 From: Beinan Date: Tue, 21 Jul 2026 00:54:22 +0000 Subject: [PATCH] ci: fix rust-cache target path and drop doctest step The rust-cache `workspaces` pointed at `crates/lance-context-core` and `python`, but this is a single root workspace whose build output lives in `./target`. Those subdirs have no target/, so the cache never captured the compiled dependencies and every run rebuilt everything from scratch (including the rocksdb C++ library). Point it at `.` so the real target dir is cached. Also drop the `Run doc tests` step: the core crate has a single doctest (~7s locally) but the step re-links the whole rocksdb-bearing crate, costing ~7min per run for near-zero coverage. Co-Authored-By: Claude Opus 4 --- .github/workflows/rust-test.yml | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/.github/workflows/rust-test.yml b/.github/workflows/rust-test.yml index 6e92b02..7a8b699 100644 --- a/.github/workflows/rust-test.yml +++ b/.github/workflows/rust-test.yml @@ -37,9 +37,7 @@ jobs: - uses: Swatinem/rust-cache@v2 with: shared-key: "lance-context-deps" - workspaces: | - crates/lance-context-core - python + workspaces: "." - name: Install dependencies run: | sudo apt update @@ -73,5 +71,3 @@ jobs: cargo test -p lance-context-master \ task_store::tests::etcd_coordinates_dedupe_claims_and_target_locks \ -- --ignored --exact - - name: Run doc tests - run: cargo test -p lance-context-core --doc