Skip to content

Add and fix Clippy lints - #1163

Merged
mdekstrand merged 5 commits into
lenskit:mainfrom
mdekstrand:tweak/fix-clippy
Jul 28, 2026
Merged

Add and fix Clippy lints#1163
mdekstrand merged 5 commits into
lenskit:mainfrom
mdekstrand:tweak/fix-clippy

Conversation

@mdekstrand

Copy link
Copy Markdown
Member

This enables Clippy lints and checks them in pre-commit.

@mdekstrand mdekstrand added this to the 2026.3 milestone Jul 28, 2026
@mdekstrand mdekstrand self-assigned this Jul 28, 2026
@mdekstrand mdekstrand added build Build, test, and platform support internals Internal infrastructure (parallelism, math, etc.) labels Jul 28, 2026
@codecov

codecov Bot commented Jul 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.33333% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.61%. Comparing base (3722c90) to head (58f7ef0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/accel/tasks/mod.rs 14.28% 6 Missing ⚠️
src/accel/als/solve.rs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1163      +/-   ##
==========================================
- Coverage   89.62%   89.61%   -0.01%     
==========================================
  Files         261      261              
  Lines       17357    17328      -29     
==========================================
- Hits        15556    15529      -27     
+ Misses       1801     1799       -2     

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

@mdekstrand
mdekstrand merged commit 58f7ef0 into lenskit:main Jul 28, 2026
41 of 42 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in LensKit Development Jul 28, 2026
@mdekstrand
mdekstrand deleted the tweak/fix-clippy branch July 28, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Build, test, and platform support internals Internal infrastructure (parallelism, math, etc.)

Projects

Development

Successfully merging this pull request may close these issues.

1 participant