added better task error handling. tasks stop executing on failure now

Cargo.toml

@@ -10,7 +10,7 @@ license = "MIT OR Apache-2.0"
 name = "cargo-matrix"
 readme = "README.md"
 repository = "/~https://github.com/rustyhorde/cargo-matrix"
-version = "0.2.0"
+version = "0.2.1"
 
 [package.metadata.binstall]
 

src/runtime/execute.rs

@@ -31,6 +31,12 @@ pub(crate) enum TaskKind {
     Test,
 }
 
+#[derive(Clone, Copy, Debug)]
+pub(crate) enum TaskResult {
+    Success,
+    Fail(i32),
+}
+
 pub(crate) struct Task {
     kind: TaskKind,
     package: String,
@@ -59,7 +65,7 @@ impl Task {
         }
     }
 
-    pub(crate) fn execute(self) -> Result<()> {
+    pub(crate) fn execute(self) -> Result<TaskResult> {
         for feature_set in self.matrix {
             match self.kind {
                 TaskKind::Build => print!("{}", Paint::cyan("    Building ").bold()),
@@ -118,11 +124,14 @@ impl Task {
                 let output = cmd.output()?;
                 if output.status.success() {
                     on_success();
+                } else {
+                    let code = output.status.code().unwrap_or(-1);
+                    return Ok(TaskResult::Fail(code));
                 }
             }
         }
 
-        Ok(())
+        Ok(TaskResult::Success)
     }
 }
 

src/runtime/mod.rs

@@ -13,7 +13,7 @@ use self::cli::{Cargo, CargoSubcommands};
 use crate::{
     config::Config,
     feature::FeatureMatrix,
-    runtime::execute::{Task, TaskKind},
+    runtime::execute::{Task, TaskKind, TaskResult},
 };
 use anyhow::{anyhow, Result};
 use cargo_metadata::{Metadata, MetadataCommand, Package};
@@ -116,7 +116,7 @@ where
 
             // Execute the task against the matricies
             for (package, matrix) in matricies {
-                Task::new(
+                let task_result = Task::new(
                     task_kind,
                     package.name.clone(),
                     matrix,
@@ -125,6 +125,10 @@ where
                     *matrix_args.dry_run(),
                 )
                 .execute()?;
+                match task_result {
+                    TaskResult::Success => continue,
+                    TaskResult::Fail(code) => return Err(anyhow!("task failed: {}", code)),
+                }
             }
         }
     }