style: cli changes

arghyadipchak · Sep 8, 2024 · de8d86f · de8d86f
1 parent 6ec6d26
commit de8d86f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 9 deletions.
diff --git a/src/cli.rs b/src/cli.rs
@@ -21,23 +21,27 @@ pub struct Cli {
 #[derive(Subcommand)]
 pub enum Commands {
   ListAssignments {
-    #[arg(short, long)]
+    #[arg(short, long, help = "Course id")]
     course_id: u64,
   },
 
   DownloadSubmissions {
-    #[arg(short, long)]
+    #[arg(short, long, help = "Course id")]
     course_id: u64,
 
-    #[arg(short, long)]
+    #[arg(short, long, help = "Assignment id")]
     assignment_id: u64,
 
-    #[arg(short, long)]
+    #[arg(
+      short,
+      long,
+      help = "Output file [default: assignment_{assignment_id}.yml]"
+    )]
     output_file: Option<PathBuf>,
   },
 
   UploadGrades {
-    #[arg(short, long)]
+    #[arg(short, long, help = "Assignment file")]
     file: PathBuf,
   },
 }
diff --git a/src/main.rs b/src/main.rs
@@ -123,10 +123,10 @@ fn download_submissions(
     })
     .collect();
 
-  let out_file = output_file.unwrap_or_else(|| {
+  let output_file = output_file.unwrap_or_else(|| {
     PathBuf::from(format!("assignment_{assignment_id}.yml"))
   });
-  let fh = File::create(&out_file)?;
+  let fh = File::create(output_file)?;
 
   write!(&fh, "{MOODY_DICLAIMER}\n\n")?;
   serde_yaml::to_writer(
@@ -139,8 +139,6 @@ fn download_submissions(
     },
   )?;
 
-  println!("{}", out_file.to_string_lossy());
-
   Ok(())
 }