Merge pull request #46550 from ikevin127/fix/proposal-police-notifica…

…tions [NoQA] FIX / ProposalPolice™ GH Actions Workflow - Graceful Fail (Solves Notifications on Fail Issue)
Expensify · Aug 1, 2024 · b0ca6ad · b0ca6ad
2 parents e36fcce + 2abbb6f
commit b0ca6ad
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/actions/javascript/proposalPoliceComment/index.js b/.github/actions/javascript/proposalPoliceComment/index.js
@@ -18061,7 +18061,9 @@ async function run() {
 }
 run().catch((error) => {
     console.error(error);
-    process.exit(1);
+    // Zero status ensures that the action is marked as successful regardless the outcome
+    // which means that no failure notification is sent to issue's subscribers
+    process.exit(0);
 });
 
 

diff --git a/.github/actions/javascript/proposalPoliceComment/proposalPoliceComment.ts b/.github/actions/javascript/proposalPoliceComment/proposalPoliceComment.ts
@@ -103,5 +103,7 @@ async function run() {
 
 run().catch((error) => {
     console.error(error);
-    process.exit(1);
+    // Zero status ensures that the action is marked as successful regardless the outcome
+    // which means that no failure notification is sent to issue's subscribers
+    process.exit(0);
 });