inotify-rs icon indicating copy to clipboard operation
inotify-rs copied to clipboard

Consider using cargo clippy in test workflows

Open i509VCB opened this issue 2 years ago • 1 comments

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.

i509VCB avatar Sep 08 '21 15:09 i509VCB

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!

hannobraun avatar Sep 14 '21 10:09 hannobraun