search

http-rs / http-types

Common types for HTTP operations
https://docs.rs/http-types
Apache License 2.0
200 stars 83 forks source link

Clippy: fix errors and enable ci #178

Closed ririsoft closed 4 years ago

ririsoft commented 4 years ago

Hello,

I am not sure about the project adherence to clippy. Reading #92 i understand it is accepted as a best practice with some exceptions such new_without_default.

I have thus fixed minor and straightforward clippy errors and warnings with a few exceptions described in the commit comment.

I have also added a github CI check to help keep the project clippy clean. Since clippy is consuming some precious minutes it can be moved to ubuntu only (as fmt checks) instead of all OSes if requested.

Cheers !

jbr commented 4 years ago

Thanks!