Skip to content

Apply clippy fixes for Rust 1.90#4423

Merged
hansl merged 2 commits intoboa-dev:mainfrom
hansl:rust-1-90
Sep 18, 2025
Merged

Apply clippy fixes for Rust 1.90#4423
hansl merged 2 commits intoboa-dev:mainfrom
hansl:rust-1-90

Conversation

@hansl
Copy link
Contributor

@hansl hansl commented Sep 18, 2025

This fixes all build and lint issues with Rust 1.90.

@hansl hansl requested a review from a team September 18, 2025 17:52
@hansl hansl changed the title Remove unnecessary parentheses Apply clippy fixes for Rust 1.90 Sep 18, 2025
@codecov
Copy link

codecov bot commented Sep 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.22%. Comparing base (6ddc2b4) to head (30ef1be).
⚠️ Report is 529 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4423      +/-   ##
==========================================
+ Coverage   47.24%   51.22%   +3.97%     
==========================================
  Files         476      502      +26     
  Lines       46892    51302    +4410     
==========================================
+ Hits        22154    26277    +4123     
- Misses      24738    25025     +287     

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hansl hansl enabled auto-merge September 18, 2025 18:15
@hansl hansl added this pull request to the merge queue Sep 18, 2025
@jedel1043 jedel1043 added the A-Internal Changes that don't modify execution behaviour label Sep 18, 2025
@jedel1043 jedel1043 added this to the next-release milestone Sep 18, 2025
Merged via the queue into boa-dev:main with commit 406973e Sep 18, 2025
16 checks passed
@hansl hansl deleted the rust-1-90 branch September 18, 2025 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Internal Changes that don't modify execution behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants