Update tracing and fix Clippy warnings

[JohnTitor]

1.56 Clippy has a weird bug that is related to macros or tracing items and it causes immoral warnings, like:

error: this looks like an `else {..}` but the `else` is missing
  --> conduit-router/src/lib.rs:70:11
   |
70 |     ) -> &mut Self {
   |           ^^^^^^^^^
   |
   = note: to remove this lint, add the missing `else` or add a new line before the next block
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#suspicious_else_formatting

It might be related to rust-lang/rust-clippy#6249, this resolves that issue by using 1.62, which is used by rustfmt CI.

Signed-off-by: Yuki Okushi [email protected]

[Turbo87]

the thing I've always wondered about: if we use a clippy from a Rust version other than our MSRV does it suggest patterns that are only possible with the newer version? or does it parse the rust-version field and know what to suggest or not?

[JohnTitor]

the thing I've always wondered about: if we use a clippy from a Rust version other than our MSRV does it suggest patterns that are only possible with the newer version? or does it parse the rust-version field and know what to suggest or not?

AFAIK Clippy works like the former, there's no functionality such as the latter. We have to explicitly ignore it when encountering it, but I guess it won't cause a major issue as this crate is passively-developed now and pinning versions should prevent warnings from increasing implicitly.

[Turbo87]

sounds reasonable 👍