concurrently 5.0.2 → 5.1.0

package/README.md

@@ -247,7 +247,7 @@ concurrently can be used programmatically by using the API documented below:
     to use when prefixing with `time`. Default: `yyyy-MM-dd HH:mm:ss.ZZZ`
 
 > Returns: a `Promise` that resolves if the run was successful (according to `successCondition` option),
-> or rejects otherwise.
+> or rejects, containing an array with the exit codes of each command that has been run.
 
 Example:
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "concurrently",
-  "version": "5.0.2",
+  "version": "5.1.0",
   "description": "Run commands concurrently",
   "main": "index.js",
   "bin": {

package/src/completion-listener.js

@@ -7,29 +7,33 @@ module.exports = class CompletionListener {
         this.scheduler = scheduler;
     }
 
-    listen(commands) {
-        const closeStreams = commands.map(command => command.close);
-        const allClosed = Rx.zip(...closeStreams);
-        return Rx.merge(...closeStreams).pipe(
-            bufferCount(closeStreams.length),
-            map(exitCodes => {
-                switch (this.successCondition) {
-                /* eslint-disable indent */
-                    case 'first':
-                        return exitCodes[0] === 0;
+    isSuccess(exitCodes) {
+        switch (this.successCondition) {
+        /* eslint-disable indent */
+            case 'first':
+                return exitCodes[0] === 0;
+
+            case 'last':
+                return exitCodes[exitCodes.length - 1] === 0;
 
-                    case 'last':
-                        return exitCodes[exitCodes.length - 1] === 0;
+            default:
+                return exitCodes.every(exitCode => exitCode === 0);
+            /* eslint-enable indent */
+        }
+    }
 
-                    default:
-                        return exitCodes.every(exitCode => exitCode === 0);
-                /* eslint-enable indent */
-                }
-            }),
-            switchMap(success => success
-                ? Rx.of(null, this.scheduler)
-                : Rx.throwError(new Error(), this.scheduler)),
-            take(1)
-        ).toPromise();
+    listen(commands) {
+        const closeStreams = commands.map(command => command.close);
+        return Rx.merge(...closeStreams)
+            .pipe(
+                bufferCount(closeStreams.length),
+                switchMap(exitCodes =>
+                    this.isSuccess(exitCodes)
+                        ? Rx.of(exitCodes, this.scheduler)
+                        : Rx.throwError(exitCodes, this.scheduler)
+                ),
+                take(1)
+            )
+            .toPromise();
     }
 };