Merge pull request #608 from Praqma/john/always-include-stdout-stderr

fix: include stdout and stderr in all Exec returns.
Praqma · May 6, 2021 · 7fa4070 · 7fa4070
2 parents 773d1b0 + 6e8ced1
commit 7fa4070
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/internal/app/command.go b/internal/app/command.go
@@ -102,11 +102,10 @@ func (c *Command) Exec() (ExitStatus, error) {
 	}
 	if err != nil {
 		res.code = 1
-		err = fmt.Errorf("failed to run %s: %w", c.Description, err)
 		if exiterr, ok := err.(*exec.ExitError); ok {
 			res.code = exiterr.ExitCode()
-			err = c.newExitError(exiterr.ExitCode(), stdout, stderr, err)
 		}
+		err = c.newExitError(res.code, stdout, stderr, err)
 	}
 	return res, err
 }