hyper icon indicating copy to clipboard operation
hyper copied to clipboard

Published 20 hours ago • verified publisher icon hyperium

Add clippy to CI

Open palango opened this issue 2 years ago • 2 comments

Adds limited clippy run to CI.

There were two clippy warnings which seemed useful, they're in the second commit. If you don't like them I'm happy to drop the commit.

Resolves #2977 Superseeds https://github.com/hyperium/hyper/pull/2979

palango avatar Feb 28 '23 23:02 palango

Thank you! I'm still surprised this doesn't catch that there is an ext.rs and ext/ folder. When I noticed those two existed, that's why I filed the original issue. Am I misunderstanding what that lint catches?

seanmonstar avatar Mar 01 '23 14:03 seanmonstar

Seems like there are some bugs in the lint: https://github.com/rust-lang/rust-clippy/issues/8094

palango avatar Mar 01 '23 14:03 palango