Skip to content

fix: resolve clippy warnings in integration matrix#170

Merged
kioku merged 2 commits intomainfrom
fix/latest-main-ci-clippy
Apr 16, 2026
Merged

fix: resolve clippy warnings in integration matrix#170
kioku merged 2 commits intomainfrom
fix/latest-main-ci-clippy

Conversation

@kioku
Copy link
Copy Markdown
Owner

@kioku kioku commented Apr 16, 2026

Summary

  • fix clippy duration_suboptimal_units findings by using larger-unit constructors for cache TTLs
  • replace sort_by closures with sort_by_key in URL query normalization and search ranking logic
  • keep behavior unchanged while removing warnings that were promoted to errors in CI

Validation

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --features integration -- -D warnings
  • cargo clippy -- -D warnings
  • cargo test

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 16, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/batch/mod.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@kioku kioku merged commit 555c24d into main Apr 16, 2026
9 checks passed
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