Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Faster CI for cargo audit tests #928

Merged
merged 5 commits into from
Aug 8, 2023
Merged

Faster CI for cargo audit tests #928

merged 5 commits into from
Aug 8, 2023

Conversation

Shnatsel
Copy link
Member

@Shnatsel Shnatsel commented Aug 8, 2023

Speeds up CI by:

  1. Running cargo test without the --release flag, avoiding compiling the code twice (tests are run with cargo run without --release)
  2. Using the sparse index instead of fetching crates.io index via git, which would cause a regression in test performance (gix and its deps such as miniz_oxide compiled in debug mode)

The Rust toolchain is bumped to v1.71 for sparse index support but MSRV is still enforced by the workspace job.

Speeds up CI by using the sparse index. MSRV is still enforced by the `workspace` job.
@Shnatsel Shnatsel changed the title Switch to Rust 1.71.0 for select jobs Switch to Rust 1.71.0 for select CI jobs Aug 8, 2023
…used - Abscissa doesn't seem to have an option to run acceptance tests with `cargo run --release`
@Shnatsel
Copy link
Member Author

Shnatsel commented Aug 8, 2023

cargo-audit / test-linux execution time drops from 13m 44s to 5m 24s.

cargo-audit / test-macos drops from 17m 15s to 8m.

cargo-audit / test-windows drops from 26m 34s to 13m 16s.

@Shnatsel Shnatsel changed the title Switch to Rust 1.71.0 for select CI jobs Faster CI for cargo audit tests Aug 8, 2023
@Shnatsel
Copy link
Member Author

Shnatsel commented Aug 8, 2023

rustsec / test on MSRV down from 14m 36s to 8m 6s

@Shnatsel Shnatsel merged commit bcb34ca into main Aug 8, 2023
15 checks passed
@Shnatsel Shnatsel deleted the faster-ci-2 branch August 8, 2023 17:34
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.

1 participant