From 5f43c21da801bba30ac6e8417487c7e32a0eff1e Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 23 Mar 2026 15:50:52 +0000 Subject: [PATCH] Revert "ci: replace global RUST_TEST_THREADS=1 with per-crate --test-threads=1 (#280)" This reverts commit fa42ef9adca03f9b793fd09e0286151cdf8b6f38. --- .github/workflows/ci.yml | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 6d1f72d6..41aaa1b3 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -149,6 +149,9 @@ jobs: # -crt-static: vite-task is shipped as a NAPI module in vite+, and musl Node # with native modules links to musl libc dynamically, so we must do the same. RUSTFLAGS: --cfg tokio_unstable -D warnings -C target-feature=-crt-static + # On musl, concurrent PTY operations can trigger SIGSEGV in musl internals. + # Run test threads sequentially to avoid the race. + RUST_TEST_THREADS: 1 steps: - name: Install Alpine dependencies shell: sh {0} @@ -172,14 +175,7 @@ jobs: corepack enable pnpm install - # Use cargo-nextest: it runs each test in its own process, avoiding - # musl's fork()-in-multithreaded-process SIGSEGV while keeping parallel - # execution (across processes instead of threads within one process). - # Exclude packages with custom test harnesses (harness = false) since - # nextest can't discover their tests; run those with cargo test instead. - - run: cargo install cargo-nextest --locked - - run: cargo nextest run --workspace --exclude vite_task_bin --exclude vite_task_plan - - run: cargo test -p vite_task_bin -p vite_task_plan + - run: cargo test fmt: name: Format and Check Deps