Small refactor by fixing clippy warnings#677
Small refactor by fixing clippy warnings#677ValentinLeTallec wants to merge 3 commits intoBoavizta:mainfrom
Conversation
jnioche
left a comment
There was a problem hiding this comment.
Thanks @ValentinLeTallec
Could you add a line to the CHANGES files as part of this PR?
See comment about version change
I can't see any clippy warnings on https://github.com/Boavizta/cloud-scanner/actions/runs/13189896898/job/36820573288
did you get them by running it on the command line? if so, should we do similar warnings as part of our actions?
| let api: BoaviztaApiV1 = BoaviztaApiV1::new(TEST_API_URL); | ||
| let version = api.get_api_version().await; | ||
| let expected = Some("1.3.7".to_owned()); | ||
| let expected = Some("1.3.8".to_owned()); |
There was a problem hiding this comment.
Let's do the version upgrade separately across the whole project.
https://github.com/Boavizta/cloud-scanner/blob/main/docker-compose.yml#L60 will need changing as well
There was a problem hiding this comment.
Registered an issue to track this upgrade separately.
#681
Clippy warning may also differ accoring to the version of the Rust toolset. @ValentinLeTallec could you please provide the output of On the laptop I'm typing this right now I have a slightly older version of the compiler (1.82), maybe this explains the difference. rustup --version
rustup 1.27.1 (54dd3d00f 2024-04-24)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active `rustc` version is `rustc 1.82.0 (f6e511eec 2024-10-15)`
cargo clippy --version
clippy 0.1.82 (f6e511e 2024-10-15) |
|
Indeed, I am on Thought to be more specific, the warning from default clippy that I have fixed are in 302e838. #![warn(
clippy::all,
clippy::pedantic,
clippy::nursery,
clippy::cargo,
clippy::unwrap_used,
clippy::expect_used,
clippy::wildcard_enum_match_arm,
clippy::use_debug
)]
#![allow(clippy::multiple_crate_versions)] // This one comes from `clippy::cargo` but is hard to resolve so I selectively allow it. |
8a247ea to
0aa795c
Compare
No description provided.