hannobraun / inotify-rs

Idiomatic inotify wrapper for the Rust programming language
ISC License
254 stars 64 forks source link

Add Clippy lints to CI #196

Closed talklittle closed 2 years ago

talklittle commented 2 years ago

Fixes #181

I also made an API deprecation, since Clippy was complaining about Event.into_owned() taking &self which is unexpected. into_owned() methods usually take self by value, not by reference. See https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention
Resolution was to deprecate into_owned() and add the properly named method, to_owned().
This change may be relevant to #178