fix: conflicting option names

README.md

@@ -1,5 +1,7 @@
 # man-completions
 
 This is an unfinished project to parse manpages to get completions for Zsh, and other shells.
+Also generates JSON files, in case your shell isn't supported, so you can process
+it and generate completions yourself.
 
 Ported from [Fish's completions script](https://github.com/fish-shell/fish-shell/blob/master/share/tools/create_manpage_completions.py)

src/main.rs

@@ -15,6 +15,7 @@ use std::process::Command;
 
 #[derive(Debug, Clone, ValueEnum)]
 enum Shell {
+  /// Generate completions for Zsh
   Zsh,
   /// Not a shell, but output the parsed options as JSON
   Json,
@@ -24,10 +25,6 @@ enum Shell {
 #[derive(Debug, Parser)]
 #[command(version, about, long_about)]
 struct CLI {
-  /// Shell to generate completions for
-  #[arg(short, long)]
-  shell: Shell,
-
   /// Directory to output completions to
   #[arg(short, long)]
   out: PathBuf,
@@ -51,7 +48,11 @@ struct CLI {
 
   /// A particular command to generate completions for. If omitted, generates
   /// completions for all found commands.
+  #[arg(short, long)]
   cmd: Option<String>,
+
+  /// Shell to generate completions for
+  shell: Shell,
 }
 
 fn section_num_parser(s: &str) -> core::result::Result<u8, String> {

tests/integration_tests.rs

@@ -27,7 +27,7 @@ fn test() {
     let cmd =
       cmd
         .env("MANPATH", &in_dir)
-        .args(["--shell", shell, "--out", &out_dir.display().to_string()]);
+        .args([shell, "--out", &out_dir.display().to_string()]);
     cmd.assert().success();
   }