Skip to content

Address new clippy lints for Rust 1.86#158

Merged
simu merged 4 commits intomainfrom
fix/clippy-1.86
Apr 4, 2025
Merged

Address new clippy lints for Rust 1.86#158
simu merged 4 commits intomainfrom
fix/clippy-1.86

Conversation

@simu
Copy link
Copy Markdown
Member

@simu simu commented Apr 3, 2025

Follow-up to #157

Checklist

  • The PR has a meaningful title. The title will be used to auto generate the changelog
  • PR contains a single logical change (to build a better changelog).
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency, internal
    as they show up in the changelog
  • Link this PR to related PRs or issues.

@github-actions

This comment was marked as outdated.

Base automatically changed from fix/flattening-errors-context to main April 3, 2025 14:08
simu added 4 commits April 3, 2025 16:11
…r`" lint

Fixing this because (quoting the clippy warning) "this will needlessly
iterate the entire iterator".
…ten token lists

This fixes clippy "warning: `format!(..)` appended to existing
`String`".
This fixes clippy "warning: `format!(..)` appended to existing
`String`".
@simu simu force-pushed the fix/clippy-1.86 branch from 133e333 to fa7baa1 Compare April 3, 2025 14:11
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 3, 2025

Benchmark for 90a6e83

Click to view benchmark
Test Base PR %
Reclass::inventory() multi-threaded 1774.1±124.37µs 1738.0±64.81µs -2.03%
Reclass::inventory() single-threaded 3.7±0.08ms 3.6±0.06ms -2.70%

@simu simu requested a review from a team April 3, 2025 14:33
@simu simu merged commit 25a7a25 into main Apr 4, 2025
22 checks passed
@simu simu deleted the fix/clippy-1.86 branch April 4, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants