fix: don't fail when result is waived

redhat-plumbers-in-action · May 22, 2024 · 53977ed · 53977ed
1 parent b2ad1ed
commit 53977ed
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 11 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/action.ts b/src/action.ts
@@ -214,12 +214,18 @@ async function action(
   err.push(...statusSummary, ...statusTables);
 
   if (err.length > 0) {
-    raise(
+    const status =
       // Show '#### Failed' header only when there is a failed message
       getFailedMessage(err, statusSummary.length > 0) +
-        '\n\n' +
-        getSuccessMessage(message)
-    );
+      '\n\n' +
+      getSuccessMessage(message);
+
+    // Don't raise error if waive label is set
+    if (isWaived) {
+      return status;
+    }
+
+    raise(status);
   }
 
   // success message only when waive label is set otherwise don't show success message only failed message