Use keyword args for optionals

Author: nozwock

src/args.rs

@@ -20,26 +20,26 @@ pub enum Commands {
     RunCustom {
         /// Testcases to run
         testcases: String,
-        #[arg(value_name = "PATH")]
+        #[arg(short, long)]
         /// File to execute
         file: Option<PathBuf>,
     },
     #[command(visible_alias = "-r")]
     Run {
-        #[arg(value_name = "PATH")]
+        #[arg(short, long)]
         /// File to execute with default testcases
         file: Option<PathBuf>,
     },
     /// Submits code to LeetCode
     #[command(visible_alias = "-fs")]
     FastSubmit {
-        #[arg(value_name = "PATH")]
+        #[arg(short, long)]
         /// File to submit
         file: Option<PathBuf>,
     },
     #[command(visible_alias = "-s")]
     Submit {
-        #[arg(value_name = "PATH")]
+        #[arg(short, long)]
         /// File to submit
         file: Option<PathBuf>,
     },
@@ -58,9 +58,10 @@ pub enum Commands {
 pub enum Execute {
     #[command(visible_alias = "-t")]
     Testcases {
-        #[arg(value_name = "PATH")]
+        #[arg(short, long)]
         /// File to run
         file: Option<PathBuf>,
+        #[arg(short, long)]
         /// Testcases to run
         testcases: Option<String>,
     },

src/main.rs

@@ -63,27 +63,24 @@ fn main() -> Result<()> {
             println!("Saved question as HTML to {}", filename.cyan());
             open::that(filename)?;
         }
-        Some(Commands::RunCustom {
-            testcases,
-            file: filename,
-        }) => {
-            _ = execute_testcases(filename, Some(&testcases), &lc)?;
+        Some(Commands::RunCustom { testcases, file }) => {
+            _ = execute_testcases(file, Some(&testcases), &lc)?;
             // bail if `is_correct == false`?
         }
         Some(Commands::Run { file }) => {
             _ = execute_testcases(file, None, &lc)?;
         }
-        Some(Commands::FastSubmit { file: filename }) => {
-            let code_file = if let Some(path) = filename {
+        Some(Commands::FastSubmit { file }) => {
+            let code_file = if let Some(path) = file {
                 CodeFile::from_file(&path)?
             } else {
                 CodeFile::from_dir(".")?
             };
 
             submit(&lc, code_file)?;
         }
-        Some(Commands::Submit { file: filename }) => {
-            let (is_correct, code_file) = execute_testcases(filename, None, &lc)?;
+        Some(Commands::Submit { file }) => {
+            let (is_correct, code_file) = execute_testcases(file, None, &lc)?;
             if is_correct {
                 submit(&lc, code_file)?;
             } else {