Merge pull request #49 from ashleygwilliams/pretty-output

fix(style): improve output messaging

Author: ashleygwilliams

Cargo.lock

@@ -8,7 +8,9 @@ license = "MIT/Apache-2.0"
 readme = "README.md"
 
 [dependencies]
+console = "0.6.1"
 failure = "0.1.1"
+indicatif = "0.9.0"
 quicli = "0.2"
 serde = "1.0"
 serde_derive = "1.0"

Cargo.toml

@@ -8,21 +8,14 @@ pub fn cargo_install_wasm_bindgen() {
         .output()
         .unwrap_or_else(|e| panic!("failed to execute process: {}", e));
 
-    if output.status.success() {
-        let s = String::from_utf8_lossy(&output.stdout);
-
-        println!(
-            "✅  cargo_install_wasm_bindgen succeeded and stdout was:\n{}",
-            s
-        );
-    } else {
-        let s = String::from_utf8_lossy(&output.stderr);
-
-        print!(
-            "⛔  cargo_install_wasm_bindgen failed and stderr was:\n{}",
-            s
-        );
-    }
+    //if !output.status.success() {
+    //    let s = String::from_utf8_lossy(&output.stderr);
+
+    //      print!(
+    //         "⛔  cargo_install_wasm_bindgen failed and stderr was:\n{}",
+    //         s
+    //     );
+    // }
 }
 
 pub fn wasm_bindgen_build(path: &str, name: &str) {
@@ -36,12 +29,7 @@ pub fn wasm_bindgen_build(path: &str, name: &str) {
         .output()
         .unwrap_or_else(|e| panic!("failed to execute process: {}", e));
 
-    if output.status.success() {
-        let s = String::from_utf8_lossy(&output.stdout);
-
-        println!("✅  wasm_bindgen_build succeeded and stdout was:\n{}", s);
-        println!("🏄<200d>♀️  ready to go!");
-    } else {
+    if !output.status.success() {
         let s = String::from_utf8_lossy(&output.stderr);
 
         print!("⛔  wasm_bindgen_build failed and stderr was:\n{}", s);

src/bindgen.rs

@@ -8,14 +8,7 @@ pub fn rustup_add_wasm_target() {
         .output()
         .unwrap_or_else(|e| panic!("failed to execute process: {}", e));
 
-    if output.status.success() {
-        let s = String::from_utf8_lossy(&output.stdout);
-
-        println!(
-            "✅ rustup_add_wasm_target succeeded and stdout was:\n{}",
-            s
-        );
-    } else {
+    if !output.status.success() {
         let s = String::from_utf8_lossy(&output.stderr);
 
         print!("⛔  rustup_add_wasm_target failed and stderr was:\n{}", s);
@@ -32,12 +25,7 @@ pub fn cargo_build_wasm(path: &str) {
         .output()
         .unwrap_or_else(|e| panic!("failed to execute process: {}", e));
 
-    if output.status.success() {
-        let s = String::from_utf8_lossy(&output.stdout);
-
-        println!("✅  cargo_build_wasm succeeded and stdout was:\n{}", s);
-        println!("🏎️ 💨  compiled to wasm!");
-    } else {
+    if !output.status.success() {
         let s = String::from_utf8_lossy(&output.stderr);
 
         print!("⛔  cargo_build_wasm failed and stderr was:\n{}", s);

src/build.rs

@@ -1,4 +1,5 @@
 extern crate failure;
+extern crate indicatif;
 #[macro_use]
 extern crate serde_derive;
 extern crate serde_json;