Merge pull request #22 from KnpLabs/v0.2.0

v0.2.0

Author: nm2107

.gitignore

@@ -2,5 +2,6 @@
 ci/Cargo.lock
 cli/Cargo.lock
 git/Cargo.lock
+logger/Cargo.lock
 utils/Cargo.lock
 target/

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "ssc"
-version = "0.1.0"
+version = "0.2.0"
 description = "A CLI tool to skip a CI build that is not concerned by the latest changes."
 homepage = "https://github.com/KnpLabs/should-skip-ci"
 repository = "git@github.com:KnpLabs/should-skip-ci.git"
@@ -15,6 +15,7 @@ edition = "2018"
 ci = { path = "ci" }
 cli = { path = "cli" }
 git = { path = "git" }
+logger = { path= "logger" }
 
 [dev-dependencies]
 rand = "0.7"

Cargo.toml

@@ -62,6 +62,8 @@ $ sudo curl -sSL -o /usr/local/bin/ssc https://github.com/KnpLabs/should-skip-ci
 $ sudo chmod +x /usr/local/bin/ssc
 ```
 
+See the latest version in the [releases panel](https://github.com/KnpLabs/should-skip-ci/releases).
+
 ### Usage
 
 ```
@@ -71,6 +73,7 @@ USAGE:
 FLAGS:
     -h, --help       Prints help information
     -V, --version    Prints version information
+    -v,-vv           Increases verbosity (-v for info, -vv for debug).
 
 OPTIONS:
     --base-branch <base-branch>    The branch to use as a base to know from where the commit range starts (i.e. to

README.md

@@ -1,5 +1,8 @@
 [package]
 name = "ci"
-version = "0.1.0"
+version = "0.2.0"
 authors = ["KNP Labs"]
 edition = "2018"
+
+[dependencies]
+log = "0.4.8"

ci/Cargo.toml

@@ -1,16 +1,26 @@
+extern crate log;
+
 use std::process::Command;
+use log::debug;
 
 pub fn run_stop_cmd(cmd: &String) -> i32 {
     // As the cmd to run is a single string, we run it though a shell
     // otherwise rust will look for an executable which *is* the whole
     // `cmd` string.
-    let result = Command::new("/bin/sh")
+    let mut command = Command::new("/bin/sh");
+    command
         .arg("-c")
         .arg(cmd)
+    ;
+
+    debug!("Running `{:?}`", command);
+
+    let result = command
         .output()
         .expect("Unable to run the stop command.")
     ;
 
+
     return result.status.code().unwrap();
 }
 

ci/src/lib.rs

@@ -1,6 +1,6 @@
 [package]
 name = "cli"
-version = "0.1.0"
+version = "0.2.0"
 authors = ["KNP Labs"]
 edition = "2018"
 

cli/Cargo.toml

@@ -17,6 +17,11 @@ struct RawCli {
 
     #[structopt(long = "cmd", help = "The command to use to skip the build.")]
     cmd: String,
+
+    // The number of occurrences of the `v/verbose` flag
+    /// Verbose mode (-v, -vv, -vvv, etc.)
+    #[structopt(short, long, parse(from_occurrences), help = "Verbosity mode : -v, -vv")]
+    verbosity: u8,
 }
 
 // The Cli struct represents the resolved CLI args and options.
@@ -57,4 +62,8 @@ impl Cli {
     pub fn cmd(&self) -> &String {
         return &self.raw_cli.cmd;
     }
+
+    pub fn verbosity(&self) -> &u8 {
+        return &self.raw_cli.verbosity;
+    }
 }

cli/src/lib.rs

@@ -1,8 +1,9 @@
 [package]
 name = "git"
-version = "0.1.0"
+version = "0.2.0"
 authors = ["KNP Labs"]
 edition = "2018"
 
 [dependencies]
 utils = { path = "../utils" }
+log = "0.4.8"

git/Cargo.toml

@@ -1,16 +1,25 @@
+extern crate log;
+
 use std::path::PathBuf;
 use std::process::Command;
+use log::debug;
 
 use utils::assert_or_panic;
 
 pub fn get_current_branch(
     working_directory: &PathBuf,
 ) -> String {
-    let result = Command::new("git")
+    let mut cmd = Command::new("git");
+    cmd
         .arg("rev-parse")
         .arg("--abbrev-ref")
         .arg("HEAD")
         .current_dir(&working_directory)
+    ;
+
+    debug!("Running `{:?}`", cmd);
+
+    let result = cmd
         .output()
         .expect("Failed to determine current branch.")
     ;
@@ -25,10 +34,16 @@ pub fn get_current_branch(
 pub fn get_current_remote(
     working_directory: &PathBuf,
 ) -> String {
-    let result = Command::new("git")
+    let mut cmd = Command::new("git");
+    cmd
         .arg("remote")
         .arg("show")
         .current_dir(&working_directory)
+    ;
+
+    debug!("Running `{:?}`", cmd);
+
+    let result = cmd
         .output()
         .expect("Failed to determine current remote.")
     ;
@@ -45,7 +60,8 @@ pub fn get_merge_base_commit(
     remote: &String,
     base_branch: &String,
 ) -> String {
-    let result = Command::new("git")
+    let mut cmd = Command::new("git");
+    cmd
         .arg("merge-base")
         .arg(format!(
             "{}/{}",
@@ -54,6 +70,11 @@ pub fn get_merge_base_commit(
         ))
         .arg("HEAD")
         .current_dir(&working_directory)
+    ;
+
+    debug!("Running `{:?}`", cmd);
+
+    let result = cmd
         .output()
         .expect("Failed to determine merge base.")
     ;