fix lit analyzer

ZebraDevs · Dec 18, 2024 · d493d43 · d493d43
1 parent 31b9b97
commit d493d43
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 32 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/scripts/analyze.ts b/src/scripts/analyze.ts
@@ -65,31 +65,33 @@ export const litAnalyzer = async (command: Command): Promise<stepResponse> => {
     });
   } catch (error) {
     setFailed(`Failed ${command.label}: ${error}`);
+    response.error = true;
   }
 
-  const lines = outputStr.split("\n");
-  const table = lines
-    .map((line) => {
-      const match = line.match(/^\s*(\S+)\s+(\d+):\s+(.*)$/);
-      if (match) {
-        const [_, file, line, message] = match;
-        return `<tr><td>${file}</td><td>${line}</td><td>${message}</td></tr>`;
-      }
-      return "";
-    })
-    .join("");
+  if (response.error == true) {
+    const lines = outputStr.split("\n");
+    const table = lines
+      .map((line) => {
+        const match = line.match(/^\s*(\S+)\s+(\d+):\s+(.*)$/);
+        if (match) {
+          const [_, file, line, message] = match;
+          return `<tr><td>${file}</td><td>${line}</td><td>${message}</td></tr>`;
+        }
+        return "";
+      })
+      .join("");
 
-  const problemCount = lines.filter((line) =>
-    line.match(/^\s*(\S+)\s+(\d+):\s+(.*)$/),
-  ).length;
+    const problemCount = lines.filter((line) =>
+      line.match(/^\s*(\S+)\s+(\d+):\s+(.*)$/),
+    ).length;
 
-  if (problemCount > 0) {
-    response.error = true;
     response.output = `${failedEmoji} - ${command.label}: ${problemCount} problem${problemCount !== 1 ? "s" : ""} found\n<details><summary>See Details</summary><table><tr><th>File</th><th>Line</th><th>Message</th></tr>${table}</table></details>`;
+
+    return response;
   } else {
     response.output = `${passedEmoji} - ${command.label}\n`;
+    return response;
   }
-  return response;
 };
 
 export const analyze = async (): Promise<stepResponse> => {