Fix forge install issues (#18) #159
clippy
5 warnings
Details
Results
Message level | Amount |
---|---|
Internal compiler error | 0 |
Error | 0 |
Warning | 5 |
Note | 0 |
Help | 0 |
Versions
- rustc 1.79.0 (129f3b996 2024-06-10)
- cargo 1.79.0 (ffa9cf99a 2024-06-03)
- clippy 0.1.79 (129f3b9 2024-06-10)
Annotations
Check warning on line 60 in daedalus_client/src/fabric.rs
github-actions / clippy
this function has too many arguments (8/7)
warning: this function has too many arguments (8/7)
--> daedalus_client/src/fabric.rs:50:1
|
50 | #[tracing::instrument(skip(semaphore, upload_files, mirror_artifacts))]
| ^----------------------------------------------------------------------
| |
| _in this procedural macro expansion
| |
51 | | async fn fetch(
52 | | format_version: usize,
53 | | mod_loader: &str,
... |
59 | | mirror_artifacts: &DashMap<String, MirrorArtifact>,
60 | | ) -> Result<(), Error> {
| |______________________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
= note: `#[warn(clippy::too_many_arguments)]` on by default
= note: this warning originates in the attribute macro `tracing::instrument` (in Nightly builds, run with -Z macro-backtrace for more info)
Check warning on line 445 in daedalus_client/src/forge.rs
github-actions / clippy
field `path` is never read
warning: field `path` is never read
--> daedalus_client/src/forge.rs:445:25
|
440 | struct ForgeInstallerProfileV2 {
| ----------------------- field in this struct
...
445 | pub path: Option<String>,
| ^^^^
|
= note: `ForgeInstallerProfileV2` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
= note: `#[warn(dead_code)]` on by default
Check warning on line 163 in daedalus_client/src/main.rs
github-actions / clippy
variable does not need to be mutable
warning: variable does not need to be mutable
--> daedalus_client/src/main.rs:163:9
|
163 | let mut val = mirror_artifacts
| ----^^^
| |
| help: remove this `mut`
|
= note: `#[warn(unused_mut)]` on by default
Check warning on line 2 in daedalus_client/src/forge.rs
github-actions / clippy
unused import: `sha1_async`
warning: unused import: `sha1_async`
--> daedalus_client/src/forge.rs:2:55
|
2 | download_file, fetch_json, fetch_xml, format_url, sha1_async,
| ^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
Check warning on line 36 in daedalus/src/modded.rs
github-actions / clippy
use of deprecated method `chrono::TimeZone::datetime_from_str`: use `DateTime::parse_from_str` or `NaiveDateTime::parse_from_str` with `and_utc()` or `and_local_timezone()` instead
warning: use of deprecated method `chrono::TimeZone::datetime_from_str`: use `DateTime::parse_from_str` or `NaiveDateTime::parse_from_str` with `and_utc()` or `and_local_timezone()` instead
--> daedalus/src/modded.rs:36:26
|
36 | .or_else(|_| Utc.datetime_from_str(&s, "%Y-%m-%dT%H:%M:%S%.9f"))
| ^^^^^^^^^^^^^^^^^
|
= note: `#[warn(deprecated)]` on by default