Skip to content

Conversation

@newpavlov
Copy link
Member

No description provided.

@newpavlov newpavlov merged commit 52425b5 into master Mar 12, 2025
11 checks passed
@newpavlov newpavlov deleted the manual_div_ceil branch March 12, 2025 16:10
@tarcieri
Copy link
Member

Hmm, why wasn't clippy catching this in CI?

@newpavlov
Copy link
Member Author

Probably because it's a nightly Clippy lint (I use Nightly in my IDE) and we use stable Clippy in CI.

@tarcieri
Copy link
Member

Really... huh, I swear I've seen this lint on stable but maybe I'm mistaken

@newpavlov
Copy link
Member Author

Hm, it was added in 1.83, but maybe not enabled by default yet? Running cargo +stable clippy does not trigger the warnings.

@tarcieri
Copy link
Member

Aah, perhaps it's being enabled by other lints like pedantic

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