Skip to content

fix(ci): install cargo-nextest via taiki-e/install-action - #113

Merged
bug-ops merged 1 commit into
mainfrom
fix/ci-nextest-locked-install
Aug 12, 2026
Merged

fix(ci): install cargo-nextest via taiki-e/install-action#113
bug-ops merged 1 commit into
mainfrom
fix/ci-nextest-locked-install

Conversation

@bug-ops

@bug-ops bug-ops commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • The test and coverage jobs installed cargo-nextest (and cargo-llvm-cov) via moonrepo/setup-rust's bins: input, which uses cargo-binstall and silently falls back to cargo install <tool> (without --locked) when the binstall fetch fails.
  • Nextest's locked-tripwire build guard rejects unlocked source installs at compile time, so any transient binstall network failure (as seen on windows-latest in run 31642553967) turns into a hard CI failure instead of a retry.
  • Switched both jobs to taiki-e/install-action, already used elsewhere in this workflow (cargo-deny, cross), which fetches prebuilt binaries directly and does not hit this fallback path.

Test plan

  • CI run on this branch passes the test matrix (ubuntu/macos/windows) and coverage job

moonrepo/setup-rust falls back to `cargo install cargo-nextest`
(without --locked) when its binstall fetch fails, which nextest
rejects at compile time via the locked-tripwire guard. Use
taiki-e/install-action, which fetches prebuilt binaries directly
and does not hit this fallback path.
@github-actions github-actions Bot added needs-review Needs review ci CI/CD workflows size: XS <10 lines changed labels Aug 12, 2026
@bug-ops
bug-ops enabled auto-merge (squash) August 12, 2026 21:50
@bug-ops
bug-ops merged commit ad52dc9 into main Aug 12, 2026
22 checks passed
@bug-ops
bug-ops deleted the fix/ci-nextest-locked-install branch August 12, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD workflows needs-review Needs review size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant