-
Notifications
You must be signed in to change notification settings - Fork 9
Small refactor by fixing clippy warnings #677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Small refactor by fixing clippy warnings #677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.