Closed Mathieu-Lala closed 2 years ago
Addressing https://github.com/dequbed/rsasl/issues/14
I also added the clippy::pedantic warnings.
clippy::pedantic
I believe most of the changes are useful, but we might want to do some cherry pick.
#[warn(clippy::{all,pedantic,exhaustive_enums,exhaustive_structs})]
lib.rs
#[allow(...)]
There is few #[allow(...)] we should fix before merging
Context
Addressing https://github.com/dequbed/rsasl/issues/14
I also added the
clippy::pedantic
warnings.I believe most of the changes are useful, but we might want to do some cherry pick.
Summary of changes
Added
#[warn(clippy::{all,pedantic,exhaustive_enums,exhaustive_structs})]
inlib.rs
#[allow(...)]
for too verbose warnings inlib.rs
(but they should be addressed)Changed
#[allow(...)]
when the fix would produce side-effect (break api)