fix: do not report with empty issues

Keeper-Wallet · Nov 16, 2022 · eebd131 · eebd131
1 parent f4a8725
commit eebd131
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 19 deletions.
diff --git a/build/index.js b/build/index.js
@@ -20000,8 +20000,12 @@ async function run() {
         const url = `${hooksUrl}/${hookMerge}`;
         const headers = { "Content-Type": "application/json" };
         const body = JSON.stringify({ issues });
-        console.log(`\u2705 Reporting issues as merged: ${issues.join(", ")}`);
-        await http.post(url, body, headers);
+        if (issues.length === 0) {
+          console.log("No issues to report");
+        } else {
+          console.log(`\u2705 Reporting issues as merged: ${issues.join(", ")}`);
+          await http.post(url, body, headers);
+        }
         break;
       }
       case "release": {
@@ -20016,12 +20020,16 @@ async function run() {
         const url = `${hooksUrl}/${hookRelease}`;
         const headers = { "Content-Type": "application/json" };
         const body = JSON.stringify({ component, issues, releaseVersion });
-        console.log(
-          `\u{1F680} Reporting issues as released in ${component} ${releaseVersion}: ${issues.join(
-            ", "
-          )}`
-        );
-        await http.post(url, body, headers);
+        if (issues.length === 0) {
+          console.log("No issues to report");
+        } else {
+          console.log(
+            `\u{1F680} Reporting issues as released in ${component} ${releaseVersion}: ${issues.join(
+              ", "
+            )}`
+          );
+          await http.post(url, body, headers);
+        }
         break;
       }
       default:

diff --git a/src/index.ts b/src/index.ts
@@ -61,9 +61,14 @@ async function run() {
         const headers = { 'Content-Type': 'application/json' };
         const body = JSON.stringify({ issues });
 
-        // eslint-disable-next-line no-console
-        console.log(`✅ Reporting issues as merged: ${issues.join(', ')}`);
-        await http.post(url, body, headers);
+        if (issues.length === 0) {
+          // eslint-disable-next-line no-console
+          console.log('No issues to report');
+        } else {
+          // eslint-disable-next-line no-console
+          console.log(`✅ Reporting issues as merged: ${issues.join(', ')}`);
+          await http.post(url, body, headers);
+        }
         break;
       }
       case 'release': {
@@ -82,14 +87,19 @@ async function run() {
         const headers = { 'Content-Type': 'application/json' };
         const body = JSON.stringify({ component, issues, releaseVersion });
 
-        // eslint-disable-next-line no-console
-        console.log(
-          `🚀 Reporting issues as released in ${component} ${releaseVersion}: ${issues.join(
-            ', '
-          )}`
-        );
-
-        await http.post(url, body, headers);
+        if (issues.length === 0) {
+          // eslint-disable-next-line no-console
+          console.log('No issues to report');
+        } else {
+          // eslint-disable-next-line no-console
+          console.log(
+            `🚀 Reporting issues as released in ${component} ${releaseVersion}: ${issues.join(
+              ', '
+            )}`
+          );
+
+          await http.post(url, body, headers);
+        }
         break;
       }
       default: