From 296d24dc558742f5b0569a481e0a9a3b4e28736a Mon Sep 17 00:00:00 2001 From: jamesgeorge007 Date: Tue, 26 Sep 2023 13:24:05 +0530 Subject: [PATCH] chore: better error logs --- dist/index.js | 29 ++++++++++++++++++++--------- index.js | 29 ++++++++++++++++++++--------- 2 files changed, 40 insertions(+), 18 deletions(-) diff --git a/dist/index.js b/dist/index.js index f89b554..e86a3bf 100644 --- a/dist/index.js +++ b/dist/index.js @@ -1840,20 +1840,31 @@ const toUrlFormat = (item) => { const exec = (cmd, args = []) => new Promise((resolve, reject) => { - const app = spawn(cmd, args, { stdio: "pipe" }); + const app = spawn(cmd, args, { stdio: "inherit" }); + let stdout = ""; - app.stdout.on("data", (data) => { - stdout = data; - }); + if (app.stdout) { + app.stdout.on("data", (data) => { + stdout = data; + }); + } + + let stderr = ""; + if (app.stderr) { + app.stderr.on("data", (data) => { + stderr = data; + }); + } + app.on("close", (code) => { if (code !== 0 && !stdout.includes("nothing to commit")) { - err = new Error(`Invalid status code: ${code}`); - err.code = code; - return reject(err); + return reject({ code, stderr }); } - return resolve(code); + + return resolve({ code, stdout }); }); - app.on("error", reject); + + app.on("error", () => reject({ code, stdout })); }); /** diff --git a/index.js b/index.js index 7b27274..e276cf0 100644 --- a/index.js +++ b/index.js @@ -57,20 +57,31 @@ const toUrlFormat = (item) => { const exec = (cmd, args = []) => new Promise((resolve, reject) => { - const app = spawn(cmd, args, { stdio: "pipe" }); + const app = spawn(cmd, args, { stdio: "inherit" }); + let stdout = ""; - app.stdout.on("data", (data) => { - stdout = data; - }); + if (app.stdout) { + app.stdout.on("data", (data) => { + stdout = data; + }); + } + + let stderr = ""; + if (app.stderr) { + app.stderr.on("data", (data) => { + stderr = data; + }); + } + app.on("close", (code) => { if (code !== 0 && !stdout.includes("nothing to commit")) { - err = new Error(`Invalid status code: ${code}`); - err.code = code; - return reject(err); + return reject({ code, stderr }); } - return resolve(code); + + return resolve({ code, stdout }); }); - app.on("error", reject); + + app.on("error", () => reject({ code, stdout })); }); /**