fix: #40 batch error should result in exitcode 1 (#41)

Farfetch · Mar 8, 2021 · 8ff81d8 · 8ff81d8
1 parent 4b587a9
commit 8ff81d8
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/core/cli/src/utils/status.ts b/core/cli/src/utils/status.ts
@@ -13,6 +13,7 @@ export class Status {
       logs: string[];
     }
   >();
+  private hasBatchErrors = false;
 
   private timeStartedByProject = new Map<string, number>();
 
@@ -30,7 +31,7 @@ export class Status {
   get hasErrors() {
     return Array.from(this.actionsByProjectName.values()).some(stat =>
       Boolean(stat.hasErrors)
-    );
+    ) || this.hasBatchErrors;
   }
 
   setActions(...actions: Action[]) {
@@ -189,6 +190,9 @@ export class Status {
           ).join(' ')
         );
       }
+      if (entry.level === 'error') {
+        this.hasBatchErrors = true;
+      }
     });
     this.onChange({ done, dynamic: [], height: 0 });
   }