NGSTACK-957 read and output stderr from eslint and prettier

netgen · Dec 17, 2024 · f0102a6 · f0102a6
1 parent 1c1f32c
commit f0102a6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/Action/JSLinter.php b/src/Action/JSLinter.php
@@ -44,9 +44,10 @@ protected function doExecute(Config $config, IO $io, Repository $repository, Con
 
             $result = $this->lintFile($file, $linterCommand, $linterOptions);
 
-            $io->write($result['output']);
-
-            if ($result['success'] !== true) {
+            if ($result['success'] === true) {
+                $io->write($result['output']);
+            } else {
+                $io->writeError(sprintf('<error>%s</error>', $result['error']));
                 $this->throwError($action, $io);
             }
         }
@@ -85,6 +86,7 @@ protected function lintFile(string $file, string $linterCommand, string $linterO
         return [
             'success' => $result->isSuccessful(),
             'output' => $result->getStdOut(),
+            'error' => $result->getStdErr(),
         ];
     }
 }
diff --git a/src/Action/JSPrettier.php b/src/Action/JSPrettier.php
@@ -44,9 +44,10 @@ protected function doExecute(Config $config, IO $io, Repository $repository, Con
 
             $result = $this->lintFile($file, $prettierCommand, $prettierOptions);
 
-            $io->write($result['output']);
-
-            if ($result['success'] !== true) {
+            if ($result['success'] === true) {
+                $io->write($result['output']);
+            } else {
+                $io->writeError(sprintf('<error>%s</error>', $result['error']));
                 $this->throwError($action, $io);
             }
         }
@@ -85,6 +86,7 @@ protected function lintFile(string $file, string $prettierCommand, string $prett
         return [
             'success' => $result->isSuccessful(),
             'output' => $result->getStdOut(),
+            'error' => $result->getStdErr(),
         ];
     }
 }