[tsmbland]

Description

Reverting the benchmarking action, as it was annoying and giving lots of false positives. For now we'll just have to do benchmarking manually/locally, as described in the docs

Fixes # (issue)

Type of change

• Bug fix (non-breaking change to fix an issue)
• New feature (non-breaking change to add functionality)
• Refactoring (non-breaking, non-functional change to improve maintainability)
• Optimization (non-breaking change to speed up the code)
• Breaking change (whatever its nature)
• Documentation (improve or add documentation)

Key checklist

• All tests pass: $ cargo test
• The documentation builds and looks OK: $ cargo doc
• Update release notes for the latest release if this PR adds a new feature or fixes a bug
  present in the previous release

Further checks

• Code is commented, particularly in hard-to-understand areas
• Tests added that prove fix is effective or that feature works

[Copilot]

🟢 Approval recommended

No active review issues were identified for the removal of automated benchmark checks in favor of the documented manual process.

No open issues 🎉

What changed in this PR

Removes automated benchmark checks from MUSE2’s CI workflow to avoid false-positive regression alerts, returning benchmarking to the documented manual/local process.

Changes:

• Removes the benchmark job, regression alerts, and automated baseline-update pull requests.
• Deletes the CI benchmark conversion script and stored results, and removes the baseline file’s pre-commit exclusion.

File	Description
benches/​convert_criterion_output.py	Deletes the Criterion-to-JSON converter used by CI.
benches/​benchmark-data.json	Deletes the stored CI benchmark baseline.
.pre-commit-config.yaml	Removes the deleted baseline file’s exclusion.
.github/​workflows/​cargo-test.yml	Removes the benchmark job while retaining test and coverage steps.

_{Review effort: Balanced}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.88%. Comparing base (1612d17) to head (73332ec).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1570   +/-   ##
=======================================
  Coverage   90.88%   90.88%           
=======================================
  Files          61       61           
  Lines        9121     9121           
  Branches     9121     9121           
=======================================
  Hits         8290     8290           
  Misses        509      509           
  Partials      322      322           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.