Skip to content

Commit

Permalink
chore: make Code Coverage build run tests in parallel
Browse files Browse the repository at this point in the history
According to:

https://github.com/taiki-e/cargo-llvm-cov#known-limitations

it's only defaulting to 1 thread, because of rustc issue:

rust-lang/rust#91092

but it seems the issue is that relatively infrequently some tests
will fail to be reported... which if fine with me if it makes
the CI faster. And they are talking about thousands of tests,
while we probably have <100.
  • Loading branch information
dpc committed Apr 15, 2023
1 parent a917e76 commit c7c9f7e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion flake.crane.nix
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,7 @@ rec {
pnameSuffix = "-lcov";
version = "0.0.1";
cargoArtifacts = workspaceDepsCov;
buildPhaseCargoCommand = "mkdir -p $out ; env RUST_LOG=info,timing=debug cargo llvm-cov --locked --workspace --profile $CARGO_PROFILE --lcov --all-targets --tests --output-path $out/lcov.info";
buildPhaseCargoCommand = "mkdir -p $out ; env RUST_LOG=info,timing=debug cargo llvm-cov --locked --workspace --profile $CARGO_PROFILE --lcov --all-targets --tests --output-path $out/lcov.info -- --test-threads=$(($(nproc) * 2))";
installPhaseCommand = "true";
nativeBuildInputs = commonArgs.nativeBuildInputs ++ [ cargo-llvm-cov ];
doCheck = false;
Expand Down

0 comments on commit c7c9f7e

Please sign in to comment.