refactor: log exit code in addition to the error message

jamesgeorge007 · Sep 28, 2023 · 934ea84 · 934ea84
1 parent ec436f8
commit 934ea84
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -1858,13 +1858,13 @@ const exec = (cmd, args = []) =>
 
     app.on("close", (code) => {
       if (code !== 0 && !stdout.includes("nothing to commit")) {
-        return reject({ code, message: stderr });
+        return reject({ code, stderr });
       }
 
-      return resolve({ code, stdout });
+      return resolve();
     });
 
-    app.on("error", () => reject({ code: 1, message: stderr }));
+    app.on("error", () => reject({ code: 1, stderr }));
   });
 
 /**
@@ -1992,7 +1992,8 @@ Toolkit.run(
       try {
         await commitFile();
       } catch (err) {
-        return tools.exit.failure(err.message);
+        const message = `Exit code: ${err.code}\n${err.stderr}`;
+        return tools.exit.failure(message);
       }
       tools.exit.success("Wrote to README");
     }
@@ -2042,7 +2043,8 @@ Toolkit.run(
     try {
       await commitFile();
     } catch (err) {
-      return tools.exit.failure(err.message);
+      const message = `Exit code: ${err.code}\n${err.stderr}`;
+      return tools.exit.failure(message);
     }
     tools.exit.success("Pushed to remote repository");
   },

diff --git a/index.js b/index.js
@@ -75,13 +75,13 @@ const exec = (cmd, args = []) =>
 
     app.on("close", (code) => {
       if (code !== 0 && !stdout.includes("nothing to commit")) {
-        return reject({ code, message: stderr });
+        return reject({ code, stderr });
       }
 
-      return resolve({ code, stdout });
+      return resolve();
     });
 
-    app.on("error", () => reject({ code: 1, message: stderr }));
+    app.on("error", () => reject({ code: 1, stderr }));
   });
 
 /**
@@ -209,7 +209,8 @@ Toolkit.run(
       try {
         await commitFile();
       } catch (err) {
-        return tools.exit.failure(err.message);
+        const message = `Exit code: ${err.code}\n${err.stderr}`;
+        return tools.exit.failure(message);
       }
       tools.exit.success("Wrote to README");
     }
@@ -259,7 +260,8 @@ Toolkit.run(
     try {
       await commitFile();
     } catch (err) {
-      return tools.exit.failure(err.message);
+      const message = `Exit code: ${err.code}\n${err.stderr}`;
+      return tools.exit.failure(message);
     }
     tools.exit.success("Pushed to remote repository");
   },