output problem not on error

ZebraDevs · Dec 13, 2024 · 58aefb4 · 58aefb4
1 parent 8edd47f
commit 58aefb4
Show file tree

Hide file tree

Showing 3 changed files with 35 additions and 44 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/main.ts b/src/main.ts
@@ -163,38 +163,34 @@ const eslint = async (command: Command): Promise<stepResponse> => {
         stdout: (data) => {
           outputStr += data.toString();
         },
-        stderr: (data) => {
-          outputStr += data.toString();
-        },
       },
     });
   } catch (error) {
     error = true;
     setFailed(`Failed ${command.label}: ${error}`);
   }
 
-  if (error) {
+  const lines = outputStr.split("\n");
+  const table = lines
+    .map((line) => {
+      const match = line.match(/^(.*?):(\d+):(\d+): (.*)$/);
+      if (match) {
+        const [_, file, line, column, message] = match;
+        return `<tr><td>${file}</td><td>${line}</td><td>${column}</td><td>${message}</td></tr>`;
+      }
+      return "";
+    })
+    .join("");
+
+  const problemCount = lines.filter((line) =>
+    line.match(/^(.*?):(\d+):(\d+): (.*)$/),
+  ).length;
+
+  if (problemCount > 0) {
     response.error = true;
-    // Parse the output to find errors and problems
-    const lines = outputStr.split("\n");
-    const table = lines
-      .map((line) => {
-        const match = line.match(/^(.*?):(\d+):(\d+): (.*)$/);
-        if (match) {
-          const [_, file, line, column, message] = match;
-          return `<tr><td>${file}</td><td>${line}</td><td>${column}</td><td>${message}</td></tr>`;
-        }
-        return "";
-      })
-      .join("");
-
-    const problemCount = lines.filter((line) =>
-      line.match(/^(.*?):(\d+):(\d+): (.*)$/),
-    ).length;
-    response.output = `<p>${problemCount} problem${problemCount !== 1 ? "s" : ""} found</p><details><summary>See Details</summary><table><tr><th>File</th><th>Line</th><th>Column</th><th>Message</th></tr>${table}</table></details>`;
-    return response;
+    response.output = `<li>${failedEmoji} - ${command.label}: ${problemCount} problem${problemCount !== 1 ? "s" : ""} found\n<details><summary>See Details</summary><table><tr><th>File</th><th>Line</th><th>Column</th><th>Message</th></tr>${table}</table></details></li>`;
   } else {
     response.output = `<li>${passedEmoji} - ${command.label}\n</li>`;
-    return response;
   }
+  return response;
 };