Skip to content

ci: free speedups#530

Merged
LegNeato merged 3 commits intomainfrom
ci_free_gains
Feb 16, 2026
Merged

ci: free speedups#530
LegNeato merged 3 commits intomainfrom
ci_free_gains

Conversation

@Firestar99
Copy link
Member

@Firestar99 Firestar99 commented Feb 16, 2026

Free speedups I found while working on #526 that don't require rust-cache. I'm also gonna use this PR as a reference point for that PR.

Results

Speedup main vs branch:

job windows mac linux
difftest main 26m 30s 15m 0s 14m 30s
difftest branch 22m 16s 20m 0s 11m 25s
difftest ~change -4m 15s +5m -3m
test main 18m 10s 15m 41s 11m 28s
test branch 22m 47s 13m 53s 12m 30s
test ~change +4m 30s -2m +1m
  • mac difftest: that runner seems to have been slow overall, likely fluke
  • mac test: surprising decrease, seems like mac runners have high variance in run times, likely fluke
  • windows test: surprisingly high but real increase compared to other OSs, likely caused by cargo on windows not reusing artifacts between test and dev profiles. Still an overall speedup in end to end, since difftest was the bottleneck

End to end: 26m 41s -> 23m 29s: -3min 10s
#526 has an end to end of 15m 41s with a cache per job, so there's still 8min to gain

@Firestar99 Firestar99 marked this pull request as ready for review February 16, 2026 15:53
@Firestar99 Firestar99 enabled auto-merge February 16, 2026 15:55
@Firestar99 Firestar99 disabled auto-merge February 16, 2026 16:21
Copy link
Collaborator

@LegNeato LegNeato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@LegNeato LegNeato added this pull request to the merge queue Feb 16, 2026
Merged via the queue into main with commit eb79ff7 Feb 16, 2026
13 checks passed
@LegNeato LegNeato deleted the ci_free_gains branch February 16, 2026 18:54
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