Skip to content

revert: undo per-crate --test-threads=1 split for musl CI#289

Merged
branchseer merged 1 commit intomainfrom
claude/revert-commit-fa42ef9-Cfh92
Mar 23, 2026
Merged

revert: undo per-crate --test-threads=1 split for musl CI#289
branchseer merged 1 commit intomainfrom
claude/revert-commit-fa42ef9-Cfh92

Conversation

@branchseer
Copy link
Member

Summary

Reverts fa42ef9 ("ci: replace global RUST_TEST_THREADS=1 with per-crate --test-threads=1 (#280)").

The change replaced the simple RUST_TEST_THREADS=1 env var with cargo-nextest installation and split test commands. This actually regressed musl CI performance:

CI Time Comparison (musl job)

Before (7a8ad8b) After (fa42ef9) Delta
Test (musl) 4m 31s 12m 32s +8m 01s (+177%)

The overhead comes from installing cargo-nextest and running two separate cargo invocations instead of one.

This revert restores the global RUST_TEST_THREADS=1 approach, bringing the musl CI job back to ~4.5 minutes.

Test plan

  • Verify musl CI job passes
  • Verify musl CI job completes in ~4-5 minutes instead of ~12 minutes

https://claude.ai/code/session_01LGzZqKECkrqipZfmmU9L3U

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@branchseer branchseer merged commit e35c8af into main Mar 23, 2026
9 checks passed
@branchseer branchseer deleted the claude/revert-commit-fa42ef9-Cfh92 branch March 23, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants