Skip to content

Conversation

@conradludgate
Copy link
Contributor

I like to run nightly clippy every so often to make our future rust upgrades easier. Some notable changes:

  • Prefer next_back() over last(). Generic iterators will implement last() to run forward through the iterator until the end.

  • Prefer io::Error::other().

  • Use implicit returns

One case where I haven't dealt with the issues is the now more-sensitive "large enum variant" lint. I chose not to take any decisions around it here, and simply marked them as allow for now.

@github-actions
Copy link

github-actions bot commented Mar 17, 2025

8019 tests run: 7635 passed, 0 failed, 384 skipped (full report)


Flaky tests (6)

Postgres 16

Postgres 15

Postgres 14

Code coverage* (full report)

  • functions: 32.3% (8734 of 27036 functions)
  • lines: 48.5% (75332 of 155198 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
c6372b1 at 2025-03-31T16:25:25.113Z :recycle:

@conradludgate conradludgate changed the title conrad/clippy 2025 03 16 chore: fix clippy lints from nightly-2025-03-16 Mar 17, 2025
@conradludgate conradludgate force-pushed the conrad/clippy-2025-03-16 branch from 62212f4 to aa41fb6 Compare March 31, 2025 12:46
@tristan957
Copy link
Member

Looks like you will need to force push. The cloud repo commit that got picked up is broken.

@tristan957 tristan957 added this pull request to the merge queue Apr 9, 2025
Merged via the queue into main with commit 72832b3 Apr 9, 2025
100 checks passed
@tristan957 tristan957 deleted the conrad/clippy-2025-03-16 branch April 9, 2025 15:12
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.

5 participants