Remove labels instead of deleting (#80)

pascalgn · Jun 4, 2020 · 4536e88 · 4536e88
1 parent d8d7bba
commit 4536e88
Showing 1 changed file with 12 additions and 15 deletions.
diff --git a/lib/merge.js b/lib/merge.js
@@ -97,23 +97,20 @@ async function removeLabels(octokit, pullRequest, mergeRemoveLabels) {
     return;
   }
 
-  logger.debug(
-    "Removing labels:",
-    labels.map(label => label.name)
-  );
+  const labelNames = labels.map(label => label.name);
 
-  await Promise.all(
-    labels.map(label =>
-      octokit.issues.deleteLabel({
-        owner: pullRequest.base.repo.owner.login,
-        repo: pullRequest.base.repo.name,
-        issue_number: pullRequest.number,
-        name: label.name
-      })
-    )
-  );
+  logger.debug("Removing labels:", labelNames);
+
+  for (const name of labelNames) {
+    await octokit.issues.removeLabel({
+      owner: pullRequest.base.repo.owner.login,
+      repo: pullRequest.base.repo.name,
+      issue_number: pullRequest.number,
+      name
+    });
+  }
 
-  logger.info("Removed labels:", labels);
+  logger.info("Removed labels:", labelNames);
 }
 
 async function deleteBranch(octokit, pullRequest) {