Skip to content

clippy: redundant clones in networking code#10571

Merged
nagisa merged 1 commit intoanza-xyz:masterfrom
nagisa:network-clones
Feb 12, 2026
Merged

clippy: redundant clones in networking code#10571
nagisa merged 1 commit intoanza-xyz:masterfrom
nagisa:network-clones

Conversation

@nagisa
Copy link
Copy Markdown

@nagisa nagisa commented Feb 12, 2026

These are detected by the clippy::redundant_clone lint which I hope to deny by default in near future.

ref. #10121

@nagisa nagisa requested a review from a team as a code owner February 12, 2026 15:15
These are detected by the `clippy::redundant_clone` lint which I hope to deny by default in near future.

ref. anza-xyz#10121
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.0%. Comparing base (79d5da0) to head (62c403c).

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #10571   +/-   ##
=======================================
  Coverage    83.0%    83.0%           
=======================================
  Files         843      843           
  Lines      315809   315805    -4     
=======================================
+ Hits       262302   262308    +6     
+ Misses      53507    53497   -10     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown

@alexpyattaev alexpyattaev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nagisa nagisa added this pull request to the merge queue Feb 12, 2026
Merged via the queue into anza-xyz:master with commit 4a5e0b9 Feb 12, 2026
50 checks passed
@nagisa nagisa deleted the network-clones branch February 12, 2026 23:24
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.

3 participants