eslint errors and warnings are now reported as fail again

Author: johan-gorter

src/code-style/formatter.ts

@@ -40,7 +40,7 @@ export function createFormatter(dependencies: {
     let options =
       (await resolveConfig(".prettierrc")) ??
       (await resolveConfig("prettier.config.js"));
-    logger.log("formatter", `formatter options: ${JSON.stringify(options)}`);
+    // logger.log("formatter", `formatter options: ${JSON.stringify(options)}`);
     let checks = await Promise.all(files.map(runFile));
     return !checks.some((c) => !c);
 

src/code-style/linter.ts

@@ -110,7 +110,7 @@ export function createLinter(dependencies: {
         running = false;
         logger.log(
           "linter",
-          response.finished.success
+          response.finished.success && errors === 0
             ? "All files are ok"
             : `${errors} Linting problems found, ${fixable} ${
                 fix ? "fixed" : "fixable"