Refactor enum display

Author: sanders41

src/config.rs

@@ -339,7 +339,7 @@ impl Config {
 
         let license_label = "License";
         if let Some(license) = config.license {
-            println!("{}: {:?}", license_label.blue(), license);
+            println!("{}: {}", license_label.blue(), license);
         } else {
             println!("{}: null", license_label.blue());
         }
@@ -379,7 +379,7 @@ impl Config {
 
         let project_manager_label = "Project Manager";
         if let Some(project_manager) = config.project_manager {
-            println!("{}: {:?}", project_manager_label.blue(), project_manager);
+            println!("{}: {}", project_manager_label.blue(), project_manager);
         } else {
             println!("{}: null", project_manager_label.blue());
         }
@@ -401,7 +401,7 @@ impl Config {
         let dependabot_schedule_label = "Dependabot Schedule";
         if let Some(dependabot_schedule) = config.dependabot_schedule {
             println!(
-                "{}: {:?}",
+                "{}: {}",
                 dependabot_schedule_label.blue(),
                 dependabot_schedule
             );
@@ -411,7 +411,7 @@ impl Config {
 
         let dependabot_day_label = "Dependabot Day";
         if let Some(dependabot_day) = config.dependabot_day {
-            println!("{}: {:?}", dependabot_day_label.blue(), dependabot_day);
+            println!("{}: {}", dependabot_day_label.blue(), dependabot_day);
         } else {
             println!("{}: null", dependabot_day_label.blue());
         }

src/project_generator.rs

@@ -213,8 +213,8 @@ fn build_latest_pre_commit_dependencies(
         hooks.par_iter_mut().for_each(|hook| {
             if hook.get_latest_version().is_err() {
                 let error_message = format!(
-                    "Error retrieving latest pre-commit version for {:?}. Using default.",
-                    hook.hook.to_string()
+                    "Error retrieving latest pre-commit version for {}. Using default.",
+                    hook.hook
                 );
                 println!("\n{}", error_message.yellow());
             }
@@ -296,8 +296,8 @@ fn build_latest_dev_dependencies(
         packages.par_iter_mut().for_each(|package| {
             if package.get_latest_version().is_err() {
                 let error_message = format!(
-                    "Error retrieving latest python package version for {:?}. Using default.",
-                    package.package.to_string()
+                    "Error retrieving latest python package version for {}. Using default.",
+                    package.package
                 );
                 println!("\n{}", error_message.yellow());
             }

src/project_info.rs

@@ -1,5 +1,8 @@
-use std::io::Write;
-use std::path::{Path, PathBuf};
+use std::{
+    fmt,
+    io::Write,
+    path::{Path, PathBuf},
+};
 
 use anyhow::{bail, Result};
 use clap::ValueEnum;
@@ -15,6 +18,16 @@ pub enum DependabotSchedule {
     Monthly,
 }
 
+impl fmt::Display for DependabotSchedule {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        match self {
+            DependabotSchedule::Daily => write!(f, "Daily"),
+            DependabotSchedule::Weekly => write!(f, "Weekly"),
+            DependabotSchedule::Monthly => write!(f, "Montly"),
+        }
+    }
+}
+
 #[derive(Clone, Debug, Deserialize, Serialize, ValueEnum)]
 pub enum Day {
     Monday,
@@ -26,20 +39,54 @@ pub enum Day {
     Sunday,
 }
 
+impl fmt::Display for Day {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        match self {
+            Day::Monday => write!(f, "Monday"),
+            Day::Tuesday => write!(f, "Tuesday"),
+            Day::Wednesday => write!(f, "Wednesday"),
+            Day::Thursday => write!(f, "Thursday"),
+            Day::Friday => write!(f, "Friday"),
+            Day::Saturday => write!(f, "Saturday"),
+            Day::Sunday => write!(f, "Sunday"),
+        }
+    }
+}
+
 #[derive(Clone, Debug, Deserialize, Serialize, ValueEnum)]
 pub enum LicenseType {
     Mit,
     Apache2,
     NoLicense,
 }
 
+impl fmt::Display for LicenseType {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        match self {
+            LicenseType::Mit => write!(f, "MIT"),
+            LicenseType::Apache2 => write!(f, "Apache 2.0"),
+            LicenseType::NoLicense => write!(f, "No License"),
+        }
+    }
+}
+
 #[derive(Clone, Debug, Deserialize, Serialize, ValueEnum)]
 pub enum ProjectManager {
     Maturin,
     Poetry,
     Setuptools,
 }
 
+impl fmt::Display for ProjectManager {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        match self {
+            ProjectManager::Maturin => write!(f, "Maturin"),
+            ProjectManager::Poetry => write!(f, "Poetry"),
+            ProjectManager::Setuptools => write!(f, "Setuptools"),
+        }
+    }
+}
+
 struct Prompt {
     prompt_text: String,
     default: Option<String>,
@@ -278,7 +325,7 @@ fn copyright_year_prompt(license: &LicenseType, default: Option<String>) -> Resu
 
     if input.is_empty() {
         bail!(format!(
-            "A copyright year is required for {:?} license",
+            "A copyright year is required for {} license",
             license
         ));
     } else {

src/rust_files.rs

@@ -20,7 +20,7 @@ fn build_latest_dependencies(min_python_version: &str, download_latest_packages:
         packages.par_iter_mut().for_each(|package| {
             if package.get_latest_version().is_err() {
                 let error_message = format!(
-                    "Error retrieving latest crate version for {:?}. Using default.",
+                    "Error retrieving latest crate version for {}. Using default.",
                     package.name
                 );
                 println!("\n{}", error_message.yellow());