From 24b3ca092ef53bed4b6d1ae234178240e6845c94 Mon Sep 17 00:00:00 2001 From: "Ian W. Remmel" <1182361+ianwremmel@users.noreply.github.com> Date: Thu, 12 Apr 2018 21:10:48 -0700 Subject: [PATCH 1/2] fix(apply): do not suppress errors A recent refactor suppressed errors in event of non-zero exit codes. This fixes that. --- src/lib/packages.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/packages.ts b/src/lib/packages.ts index 7f77323c..d2290a04 100644 --- a/src/lib/packages.ts +++ b/src/lib/packages.ts @@ -49,8 +49,8 @@ export async function apply( await fn(packageName); log(options, debug, afterEach(packageName)); } catch (err) { + errors.push(err); if (options.failFast) { - errors.push(err); log(options, debug, afterEach(packageName, err)); } } From 8772d754ec8f34b4a43efba7dde009b6725f2302 Mon Sep 17 00:00:00 2001 From: "Ian W. Remmel" <1182361+ianwremmel@users.noreply.github.com> Date: Thu, 12 Apr 2018 21:20:19 -0700 Subject: [PATCH 2/2] fix(apply): reduce noise when subtasks fail Fixes #77 --- src/lib/packages.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/packages.ts b/src/lib/packages.ts index d2290a04..168ebea3 100644 --- a/src/lib/packages.ts +++ b/src/lib/packages.ts @@ -58,7 +58,7 @@ export async function apply( log(options, debug, after(packages, errors)); if (errors.length) { - console.error(errors); + errors.forEach((e) => console.error(e.toString())); process.exit(1); } }