chore(ci): disable CodSpeed performance analysis - #3575
Conversation
Drop the `benchmarks-instrumented` job (and its entry in the `ci-failure-alert` aggregator) and clean up stale references in the cargo-tests reusable workflow. CodSpeed is being replaced with an in-house benchmarking system. The `codspeed-divan-compat` dev-deps are intentionally left in place — they behave as a drop-in `divan` outside `cargo codspeed` builds, so local benchmarks still work. They can be swapped back to plain `divan` when the replacement system lands.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThis PR removes the ChangesCI Job and Failure Alert Removal
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Binary size checks passed✅ 7 passed
Generated by |
Summary
benchmarks-instrumentedjob fromci.yamland remove it from theci-failure-alertaggregator (needs list + summary table).benchmarks-instrumented/ "codspeed builds" references incargo-tests.reusable.yaml's cache-strategy comment.The
codspeed-divan-compatdev-deps inbaml_language/Cargo.tomlandengine/baml-vm/Cargo.tomlare intentionally kept — they behave as a drop-indivanoutsidecargo codspeedbuilds, so local benchmarks still work. They can be swapped back to plaindivanwhen the replacement lands.Test plan
benchmarks-instrumentedjob and withoutci-failure-alertreferencing it.cargo bench/cargo test --benchesinbaml_languagestill compiles and runs (sanity check that thecodspeed-divan-compatdrop-in still works without CodSpeed instrumentation).Summary by CodeRabbit