Merge pull request #6 from jnises/cleanup

Cleanup

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "git-suggest-ownership"
 authors = ["Joel Nises <[email protected]>"]
-version = "0.4.0"
+version = "0.5.0"
 edition = "2021"
 
 [profile.release]

src/main.rs

@@ -18,7 +18,7 @@ use thread_local::ThreadLocal;
 #[command(version, about, long_about = None)]
 struct Opt {
     /// Start with the files with the smallest percentage
-    #[arg(short, long, conflicts_with_all = &["show_authors", "max_authors"])]
+    #[arg(long, conflicts_with_all = &["show_authors", "max_authors"])]
     reverse: bool,
 
     /// Verbose mode (-v, -vv, -vvv, etc), disables progress bar
@@ -30,7 +30,7 @@ struct Opt {
     no_progress: bool,
 
     /// Include all files, even the ones with no lines changed by you
-    #[arg(short, long, conflicts_with_all = &["show_authors", "max_authors"])]
+    #[arg(long, conflicts_with_all = &["show_authors", "max_authors"])]
     all: bool,
 
     /// Your email address. You can specify multiple. Defaults to your configured `config.email`
@@ -49,7 +49,7 @@ struct Opt {
     max_authors: u32,
 
     /// Limit to the specified directory. Defaults to the entire repo.
-    #[arg(long)]
+    #[arg(short, long)]
     dir: Option<PathBuf>,
     // TODO add option to limit the depth of tree printed
 }
@@ -371,7 +371,7 @@ fn main() -> Result<()> {
         .par_iter()
         .filter_map(|path| {
             debug!("blaming {}", path.display());
-            let repo = repo_tls.get_or_try(&get_repo).expect("unable to get repo");
+            let repo = repo_tls.get_or_try(get_repo).expect("unable to get repo");
             let contributions = Contributions::try_from_path(repo, path);
             progress.inc(1);
             let contributions = match contributions {