style: cargo clippy

Author: ChecksumDev

src/commands/build.rs

@@ -3,7 +3,7 @@ use std::path::PathBuf;
 use crate::{api::Client, config::Config, solution::Solution};
 use anyhow::Result;
 
-pub fn build(client: Client, config: &mut Config) -> Result<()> {
+pub fn build(_client: Client, _config: &mut Config) -> Result<()> {
     println!("Building project...");
     let cwd = std::env::current_dir()?;
     let manifest_paths = manifest_search(&cwd)?;

src/commands/login.rs

@@ -1,6 +1,5 @@
 use crate::{api::Client, config::Config, utils::progress::*};
 use anyhow::Result;
-use std::time::Duration;
 use inquire::{validator::ExactLengthValidator, Password, PasswordDisplayMode, Select};
 
 enum LoginMethod {

src/commands/new.rs

@@ -1,13 +1,19 @@
 use crate::{
     api::Client,
     config::Config,
-    helpers::forge_generator::{self, ForgeGenerator},
-    utils::{progress::{make_progress, finish_progress}, get_instance_paths, get_game_version},
+    helpers::forge_generator::ForgeGenerator,
     structs::Instance,
+    utils::{
+        get_game_version, get_instance_paths,
+        progress::{finish_progress, make_progress},
+    },
 };
 use anyhow::Result;
 use convert_case::{Case, Casing};
-use inquire::{Confirm, MultiSelect, Select, Text, validator::{self, MinLengthValidator, Validation}, CustomUserError};
+use inquire::{
+    validator::{self, MinLengthValidator, Validation},
+    Confirm, CustomUserError, MultiSelect, Select, Text,
+};
 use slug::slugify;
 use std::path::PathBuf;
 
@@ -82,7 +88,7 @@ pub fn new(client: Client, _config: &mut Config) -> Result<()> {
     .prompt()?;
 
     let instances = get_instance_paths();
-    let instance = if instances.len() == 0 {
+    let instance = if instances.is_empty() {
         let ipath: PathBuf = Text::new("Could not autodetect Beat Saber install. Please enter the path to your Beat Saber install.")
             .with_validator(FileExistsValidatior)
             .prompt()?.into();
@@ -92,16 +98,24 @@ pub fn new(client: Client, _config: &mut Config) -> Result<()> {
             name: "Custom".into(),
             game_version: get_game_version(ipath.to_str().unwrap().to_string()),
         }
-
     } else {
-        Select::new("Which Beat Saber install would you like to build in?", instances).prompt()?
+        Select::new(
+            "Which Beat Saber install would you like to build in?",
+            instances,
+        )
+        .prompt()?
     };
 
     let pb = make_progress();
     pb.set_message(format!("Creating {}...", mod_name_pascal));
 
     let mod_path = std::env::current_dir()?.join(&mod_name_pascal);
-    ForgeGenerator::new(mod_name_pascal, mod_path.to_str().unwrap().to_string(), instance).generate();
+    ForgeGenerator::new(
+        mod_name_pascal,
+        mod_path.to_str().unwrap().to_string(),
+        instance,
+    )
+    .generate();
 
     finish_progress(&pb, "Done!");
 

src/commands/publish.rs

@@ -0,0 +1 @@
+

src/helpers/forge_generator.rs

@@ -4,7 +4,7 @@
 //? - checksum, sargon64 - 2023
 
 use crate::structs::Instance;
-use forge_lib::structs::{forgemod, manifest::ForgeManifest, v1::ManifestBuilder};
+use forge_lib::structs::v1::ManifestBuilder;
 use semver::{Version, VersionReq};
 use uuid::Uuid;
 
@@ -69,7 +69,8 @@ impl ForgeGenerator {
         std::fs::write(
             format!("{}/{}/beatforge.manifest", self.path, self.name),
             self.make_bf_manifest(),
-        ).unwrap();
+        )
+        .unwrap();
 
         std::fs::write(
             format!("{}/{}/{}.csproj.user", self.path, self.name, self.name),

src/helpers/mod.rs

@@ -1 +1 @@
-pub(crate) mod forge_generator;
+pub(crate) mod forge_generator;

src/main.rs

@@ -2,14 +2,14 @@ mod api;
 mod commands;
 mod config;
 mod helpers;
+mod solution;
 mod structs;
 mod utils;
-mod solution;
 
+use anyhow::{Error, Result};
 use api::Client;
 use clap::{Parser, Subcommand};
-use commands::{login::login, new::new, build::build};
-use anyhow::{Result, Error};
+use commands::{build::build, login::login, new::new};
 
 #[derive(Parser, Debug)]
 #[clap(version = env!("CARGO_PKG_VERSION"), author = "BeatForge")]

src/solution.rs

@@ -1,16 +1,19 @@
 // A struct representing a C# solution with build helpers
 
+use anyhow::Result;
 use std::{path::PathBuf, process::Command};
-use anyhow::{Result, Error};
 
 pub struct Solution {
     pub name: String,
     pub path: PathBuf,
 }
 
 impl Solution {
-    pub fn new<T: Into<String>, U: Into<PathBuf>>(name: T, path: U) -> Solution { 
-        Solution { name: name.into(), path: path.into() }
+    pub fn new<T: Into<String>, U: Into<PathBuf>>(name: T, path: U) -> Solution {
+        Solution {
+            name: name.into(),
+            path: path.into(),
+        }
     }
 
     pub fn restore(&self) -> Result<()> {

src/structs.rs

@@ -47,4 +47,4 @@ impl std::fmt::Display for Instance {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         write!(f, "{} ({})", self.name, self.game_version)
     }
-}
+}

src/utils.rs

@@ -127,7 +127,7 @@ pub fn get_game_version(path: String) -> String {
     let mut bytes = Vec::new();
     file.read_to_end(&mut bytes).unwrap();
 
-    let out = String::from_utf8_lossy(&*bytes);
+    let out = String::from_utf8_lossy(&bytes);
     let pos = out.find("public.app-category.games").unwrap();
     let regex = Regex::new(r"[\d]+.[\d]+.[\d]+(p1)?").unwrap();