Repository navigation
chore: bump codspeed gha to v5 - #3014
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are recorded for the CodSpeed v5 upgrade and benchmark output visibility changes.
No open issues 🎉
What changed in this PR
Updates the benchmark workflow to CodSpeed action v5 to address reported silent failures and improve benchmark accuracy.
Changes:
- Upgrades
CodSpeedHQ/actionfrom v4 to v5. - Removes stdout suppression from
cargo codspeed run, making benchmark output visible in workflow logs.
| File | Description |
|---|---|
.github/workflows/benchmarks.yml |
Upgrades the CodSpeed action and exposes benchmark command output. |
Review effort: Balanced
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3014 +/- ##
==========================================
+ Coverage 78.80% 78.81% +0.01%
==========================================
Files 75 75
Lines 45355 45355
==========================================
+ Hits 35740 35746 +6
+ Misses 9615 9609 -6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Hooray! CodSpeed harness just leveled up!The base and head of this comparison were measured with different runner settings, so their benchmark values are not directly comparable. What changed between base and head:
Re-run the base with the same settings to get a valid performance comparison. Comparing |
|
Thanks for your PR |
Previously this kept failing silently, the newer version resolves that and also provides for higher bench accuracy.
Prequel to #3009 and #2995 and #3012 and #3013