let code compile

Author: Sk7Str1p3

src/cli.rs

@@ -220,7 +220,7 @@ pub struct VisualsCliOptions {
 pub struct ConfigCliOptions {
     /// Path to the config file
     #[arg(long, value_hint = ValueHint::FilePath)]
-    pub config_path: PathBuf,
+    pub config_path: Option<PathBuf>,
     /// Creates a default config file
     /// Writes to $XDG_CONFIG_HOME/onefetch/config.toml but can be overridden with --config-path
     #[arg(long)]
@@ -321,7 +321,7 @@ impl Default for ConfigCliOptions {
     fn default() -> Self {
         ConfigCliOptions {
             // Not sure about unwrap
-            config_path: dirs::config_dir().unwrap().join("onefetch/config.toml"),
+            config_path: Some(dirs::config_dir().unwrap().join("onefetch/config.toml")),
             generate_config: false,
         }
     }

src/config.rs

@@ -39,16 +39,23 @@ impl ConfigOptions {
     // I mean to write disabled_fields with --disabled-fields flag
     where
         // Nah this feels really bad but rust-analyser told me to add ' + std::fmt::Display'
-        P: AsRef<Path> + std::fmt::Display,
+        P: AsRef<Path>,
     {
         // I dont think this can panic so i simply unwrapped it
         let defaults = toml::to_string(&Self::default()).unwrap();
         match fs::create_dir_all(&path) {
             Ok(_) => match fs::write(&path, &defaults) {
-                Ok(_) => println!("Default config file created at: {path}"),
-                Err(e) => eprintln!("Failed to write default config file: {}", e),
+                Ok(_) => {
+                    let path = path.as_ref().display();
+                    println!("Default config file created at: {path}")
+                }
+                Err(e) => {
+                    let path = path.as_ref().display();
+                    eprintln!("Failed to write default config file at {path}: {e}")
+                }
             },
             Err(e) => {
+                let path = path.as_ref().display();
                 eprintln!("Failed to create config directory {path}: {e}");
             }
         }

src/main.rs

@@ -32,12 +32,10 @@ fn main() -> Result<()> {
     }
 
     if cli_options.config.generate_config {
-        // @spenserblack fixme pls
-        // it complans if i add .display(), and complains if i remove .display()
-        return ConfigOptions::write_default(cli_options.config.config_path.display());
+        return ConfigOptions::write_default(cli_options.config.config_path.unwrap_or_default());
     }
 
-    let config_options = ConfigOptions::read(&cli_options.config.config_path);
+    let config_options = ConfigOptions::read(&cli_options.config.config_path.clone().unwrap_or_default());
 
     let info = build_info(&cli_options, &config_options)?;