Skip to content

chore: resolve new clippy warnings - #1066

Merged
siriak merged 1 commit into
TheAlgorithms:masterfrom
vil02:fix_new_clippy_warnings
Aug 27, 2026
Merged

chore: resolve new clippy warnings#1066
siriak merged 1 commit into
TheAlgorithms:masterfrom
vil02:fix_new_clippy_warnings

Conversation

@vil02

@vil02 vil02 commented Aug 27, 2026

Copy link
Copy Markdown
Member

Pull Request Template

Description

This PR resolves new warnings.
Similar to:

Checklist:

  • I ran bellow commands using the latest version of rust nightly.
  • I ran cargo clippy --all -- -D warnings just before my last commit and fixed any issue that was found.
  • I ran cargo fmt just before my last commit.
  • I ran cargo test just before my last commit and all tests passed.
  • I added my algorithm to the corresponding mod.rs file within its own folder, and in any parent folder(s).
  • I added my algorithm to DIRECTORY.md with the correct link.
  • I checked COUNTRIBUTING.md and my code follows its guidelines.

Please make sure that if there is a test that takes too long to run ( > 300ms), you #[ignore] that or
try to optimize your code or make the test easier to run. We have this rule because we have hundreds of
tests to run; If each one of them took 300ms, we would have to wait for a long time.

@vil02
vil02 force-pushed the fix_new_clippy_warnings branch from 458e160 to d6db993 Compare August 27, 2026 19:00
@codecov-commenter

codecov-commenter commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.86%. Comparing base (2c53ddf) to head (15a01a7).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1066      +/-   ##
==========================================
- Coverage   95.89%   95.86%   -0.04%     
==========================================
  Files         396      396              
  Lines       30440    30440              
==========================================
- Hits        29190    29180      -10     
- Misses       1250     1260      +10     

☔ 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.

@vil02
vil02 force-pushed the fix_new_clippy_warnings branch from d6db993 to 7c84f64 Compare August 27, 2026 19:03
@vil02
vil02 force-pushed the fix_new_clippy_warnings branch from 7c84f64 to 15a01a7 Compare August 27, 2026 19:05
@vil02
vil02 requested a review from siriak August 27, 2026 19:12
@vil02
vil02 marked this pull request as ready for review August 27, 2026 19:13
@vil02
vil02 requested a review from imp2002 as a code owner August 27, 2026 19:13
@siriak
siriak merged commit 2345c66 into TheAlgorithms:master Aug 27, 2026
7 checks passed
@vil02
vil02 deleted the fix_new_clippy_warnings branch August 27, 2026 20:11
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