Skip to content

chore: remove clippy msrv#573

Merged
prestwich merged 1 commit intorecmo:mainfrom
DaniPopes:clippies2
Apr 24, 2026
Merged

chore: remove clippy msrv#573
prestwich merged 1 commit intorecmo:mainfrom
DaniPopes:clippies2

Conversation

@DaniPopes
Copy link
Copy Markdown
Contributor

@DaniPopes DaniPopes commented Apr 23, 2026

Clippy MSRV is out of sync. It defaults to project Rust version so we can just remove the file.

@DaniPopes DaniPopes requested a review from prestwich as a code owner April 23, 2026 21:26
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Apr 23, 2026

Merging this PR will degrade performance by 45.99%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 3 improved benchmarks
❌ 2 regressed benchmarks
✅ 381 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
most_significant_bits/4096/4096 28.9 µs 26.1 µs +10.87%
parse/decimal/384/max 5.2 ms 8.3 ms -37.6%
parse/decimal/384/zero 486 µs 899.8 µs -45.99%
to/f32/4096 29.1 µs 26.3 µs +10.97%
to/f64/4096 29.3 µs 26.3 µs +11.17%

Comparing DaniPopes:clippies2 (2e2e920) with main (a21eb25)

Open in CodSpeed

@prestwich prestwich merged commit bff85c8 into recmo:main Apr 24, 2026
18 of 19 checks passed
@DaniPopes DaniPopes deleted the clippies2 branch April 24, 2026 20:28
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