chore: re-throw errors as AggregateErrors

Keeps error stack traces.
nikku · Feb 10, 2021 · 12f22f9 · 12f22f9
1 parent 1980083
commit 12f22f9
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/lib/core.js b/lib/core.js
@@ -2,6 +2,8 @@ const {
   getOctokit
 } = require('./recorder');
 
+const AggregateError = require('aggregate-error');
+
 const DEFAULT_MIN_APPROVALS = 1;
 
 const CONFIG_FILE = 'merge-me.yml';
@@ -274,7 +276,7 @@ async function getCollaboratorReviews(context, pullRequest) {
         continue;
       }
 
-      throw error;
+      throw new AggregateError([ error ]);
     }
   }
 
@@ -687,7 +689,7 @@ async function merge(context, pullRequest) {
 
       return false;
     } else {
-      throw error;
+      throw new AggregateError([ error ]);
     }
   }
 }
@@ -720,7 +722,7 @@ async function checkMerge(context, pullRequest) {
     } else if (isMergeCheckError(error)) {
       context.log.debug(ctx, `skipping: ${error.message}`);
     } else {
-      throw error;
+      throw new AggregateError([ error ]);
     }
   }
 
@@ -950,7 +952,7 @@ async function getTeamsWithMembers(context, pullRequest, reviewTeams) {
         throw MergeCheckError(`failed to fetch team ${teamSlug}`);
       }
 
-      throw error;
+      throw new AggregateError([ error ]);
     });
 
     return members;

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -24,6 +24,7 @@
     "auto-test": "nodemon --exec npm test"
   },
   "dependencies": {
+    "aggregate-error": "^3.1.0",
     "probot": "^10.19.0"
   },
   "devDependencies": {