Skip to content

chore: Use nextest in CI#758

Merged
Veykril merged 1 commit intosalsa-rs:masterfrom
Veykril:veykril/push-pmwrzkrmusmu
Mar 15, 2025
Merged

chore: Use nextest in CI#758
Veykril merged 1 commit intosalsa-rs:masterfrom
Veykril:veykril/push-pmwrzkrmusmu

Conversation

@Veykril
Copy link
Member

@Veykril Veykril commented Mar 15, 2025

Also adjusts some numbers in tests to have miri do less repetitive work

This more than halves the runtime of our miri job, in part due to nextest itself and in part due to the number adjusting (which I noticed due nextest complaining about long >60s running tests)

@netlify
Copy link

netlify bot commented Mar 15, 2025

Deploy Preview for salsa-rs canceled.

Name Link
🔨 Latest commit 57e9e93
🔍 Latest deploy log https://app.netlify.com/sites/salsa-rs/deploys/67d554b6f710a90008761b5f

@Veykril Veykril force-pushed the veykril/push-pmwrzkrmusmu branch 2 times, most recently from cfa126f to 4174934 Compare March 15, 2025 09:02
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 15, 2025

CodSpeed Performance Report

Merging #758 will improve performances by 8.11%

Comparing Veykril:veykril/push-pmwrzkrmusmu (57e9e93) with master (2308261)

Summary

⚡ 1 improvements
✅ 11 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
converge_diverge 170.1 µs 157.4 µs +8.11%

@Veykril Veykril force-pushed the veykril/push-pmwrzkrmusmu branch from 4174934 to 28a4e9c Compare March 15, 2025 09:19
@Veykril Veykril changed the title chore: Use nextest for miri test runs chore: Use nextest CI Mar 15, 2025
@Veykril Veykril marked this pull request as ready for review March 15, 2025 09:19
@Veykril Veykril force-pushed the veykril/push-pmwrzkrmusmu branch 3 times, most recently from cefa397 to e101767 Compare March 15, 2025 09:34
@Veykril Veykril changed the title chore: Use nextest CI chore: Use nextest in CI Mar 15, 2025
@Veykril Veykril force-pushed the veykril/push-pmwrzkrmusmu branch 6 times, most recently from 0fe6bd8 to 038412b Compare March 15, 2025 10:08
@Veykril Veykril force-pushed the veykril/push-pmwrzkrmusmu branch from 038412b to 57e9e93 Compare March 15, 2025 10:21
@Veykril Veykril enabled auto-merge March 15, 2025 10:21
@Veykril Veykril added this pull request to the merge queue Mar 15, 2025
Merged via the queue into salsa-rs:master with commit 1dc2438 Mar 15, 2025
11 checks passed
@Veykril Veykril deleted the veykril/push-pmwrzkrmusmu branch March 15, 2025 10:40
@github-actions github-actions bot mentioned this pull request Mar 15, 2025
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