Continue removing labels with errors (#237)

actions-ecosystem · May 23, 2021 · 335648f · 335648f
1 parent b5db75d
commit 335648f
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 12 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -952,13 +952,23 @@ function run() {
                 return;
             }
             const client = github.getOctokit(githubToken);
+            const remaining = [];
             for (const label of labels) {
-                yield client.issues.removeLabel({
-                    name: label,
-                    owner,
-                    repo,
-                    issue_number: number
-                });
+                try {
+                    yield client.issues.removeLabel({
+                        name: label,
+                        owner,
+                        repo,
+                        issue_number: number
+                    });
+                }
+                catch (e) {
+                    core.warning(`failed to remove label: ${label}: ${e}`);
+                    remaining.push(label);
+                }
+            }
+            if (remaining.length) {
+                throw new Error(`failed to remove labels: ${remaining}`);
             }
         }
         catch (e) {

diff --git a/src/main.ts b/src/main.ts
@@ -21,13 +21,23 @@ async function run(): Promise<void> {
 
     const client = github.getOctokit(githubToken);
 
+    const remaining = [];
     for (const label of labels) {
-      await client.issues.removeLabel({
-        name: label,
-        owner,
-        repo,
-        issue_number: number
-      });
+      try {
+        await client.issues.removeLabel({
+          name: label,
+          owner,
+          repo,
+          issue_number: number
+        });
+      } catch (e) {
+        core.warning(`failed to remove label: ${label}: ${e}`);
+        remaining.push(label);
+      }
+    }
+
+    if (remaining.length) {
+      throw new Error(`failed to remove labels: ${remaining}`);
     }
   } catch (e) {
     core.error(e);