use structopt instead of clap

many dependencies, but also convenience and safety

Author: Byron

Diff for: Cargo.lock

@@ -13,9 +13,9 @@ test = false
 doctest = false
 
 [dependencies]
-clap = "2.31.2"
 git-core = { version = "0.1.0", path = "git-core" }
 anyhow = "1.0.31"
+structopt = "0.3.14"
 
 [profile.release]
 panic = 'unwind'

Diff for: Cargo.toml

@@ -1,29 +1,32 @@
-#[macro_use]
-extern crate clap;
-extern crate git_core as git;
 use anyhow::{Context, Result};
+use git_core;
+use structopt::StructOpt;
 
-mod app {
-    use clap::{App, AppSettings, SubCommand};
+mod options {
+    use structopt::StructOpt;
 
-    pub fn new<'a, 'b>() -> App<'a, 'b> {
-        let app: App = app_from_crate!();
-        app.setting(AppSettings::SubcommandRequired).subcommand(
-            SubCommand::with_name("init")
-                .alias("initialize")
-                .about("Initialize the repository in the current directory."),
-        )
+    #[derive(Debug, StructOpt)]
+    #[structopt(about = "The git, simply swift")]
+    #[structopt(setting = structopt::clap::AppSettings::SubcommandRequired)]
+    pub struct Args {
+        #[structopt(subcommand)]
+        pub cmd: Subcommands,
+    }
+
+    #[derive(Debug, StructOpt)]
+    pub enum Subcommands {
+        /// Initialize the repository in the current directory.
+        #[structopt(alias = "initialize")]
+        Init,
     }
 }
 
 fn main() -> Result<()> {
-    let app = app::new();
-    let matches = app.get_matches();
-    match matches.subcommand() {
-        ("init", Some(_args)) => {
-            git::init::repository().with_context(|| "Repository initialization failed")
+    let args = options::Args::from_args();
+    match args.cmd {
+        options::Subcommands::Init => {
+            git_core::init::repository().with_context(|| "Repository initialization failed")
         }
-        _ => unreachable!(),
     }?;
     Ok(())
 }

Diff for: src/main.rs

@@ -0,0 +1,8 @@
+### TODO
+
+* [x] remove failure and replace with quick-check/anyhow
+* [x] minimize dependencies and test-runs
+* [ ] structopt for grit
+* [ ] make parser tests unit tests that are close to the code they test
+* [ ] parser tests handle binary strings nicely
+* [ ] switch parsing to nom (binary only)