twitch-rs / twitch_api

Rust library for talking with the Twitch API aka. "Helix", TMI and more! Use Twitch endpoints fearlessly!
Apache License 2.0
151 stars 33 forks source link

fix(clippy): resolve some new clippy warnings #431

Closed Nerixyz closed 1 month ago

Nerixyz commented 1 month ago

This PR resolves some clippy warnings. #![cfg_attr(test, allow(deprecated))] was added in lib.rs to make editing with rust-analyzer easier. Adding #![allow(deprecated)] for the pubsub module doesn't work, because of the test macros (playground).

The only remaining warnings are from too_long_first_doc_paragraph in top-level doc-comments (I don't know how to silence these, as #[allow(...)] doesn't work :/).

Other than that, twitch_types still has some warnings. These have been fixed, but there's no release yet (also not really needed, imo).

https://github.com/twitch-rs/twitch_api/pull/419 was partly done in https://github.com/twitch-rs/twitch_api/pull/425 (only the edit in Cargo.toml was left), so I did it here. Closes https://github.com/twitch-rs/twitch_api/issues/410.