chore: Add clippy to ci

[tottoto]

Adds clippy check to ci.

[LucioFranco]

I am almost more inclined to not include clippy in ci but to accept clippy PRs as them come. Mostly this is due to my frustrations of new versions of clippy adding new lints such that it blocks PRs that add unrelated things.

[tottoto]

That makes sense. I understand your point and agree with you. So how do you think about lock the clippy version to the msrv for minimal lint (if it is feasable). Is it acceptable?

[tottoto]

I'll try to implement it.

[LucioFranco]

Yeah, I am ok to lock the clippy version so that new code get caught but old code doesn't cause issues when a new version comes out.

[LucioFranco]

on a sidenote are you on discord in the tokio server?

[tottoto]

I have to fix this warning.

warning: unknown lint: `clippy::derive_partial_eq_without_eq`
 --> tests/root-crate-path/src/main.rs:1:9
  |
1 | #[allow(clippy::derive_partial_eq_without_eq)]
  |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
  = note: `#[warn(unknown_lints)]` on by default

[tottoto]

Ignores unknown lints and makes job fail when clippy reports warnings.

[tottoto]

Done. I think this is ready to be reviewed.

[tottoto]

on a sidenote are you on discord in the tokio server?

I'm not on there.