Consider using cargo clippy in test workflows

hannobraun / inotify-rs

Idiomatic inotify wrapper for the Rust programming language

ISC License

254 stars 64 forks source link

Consider using cargo clippy in test workflows #181

Closed i509VCB closed 2 years ago

i509VCB commented 3 years ago

Clippy adds quite a few interesting lints that may be useful to have.

At the moment running clippy has 4 errors, mostly redundant field names in initialization. Clippy does also handle things such as warning you about unnecessary clones, redundant borrows on an already borrowed field, etc.

hannobraun commented 3 years ago

Thank you for opening this issue, @i509VCB. I'm all for that! It's just an issue of someone doing the work, i.e. adding Clippy to the CI build and fixing the existing warnings.

Pull requests welcome!