Merge pull request #4 from jerone/bugfix/linter

Author: jerone

.config/cspell-words.txt

@@ -1,8 +1,14 @@
+Dsonar
 fileoverview
 issuetracker
 Jeroen
+jerone
 linkify
 pjax
 setext
+sonarcloud
+sonarlint
+sonarsource
 sublist
+temurin
 Warmerdam

scripts/lint/index.cjs

@@ -24,6 +24,8 @@ const {
 
 const enableDebug = false;
 
+let hasError = false;
+
 function assert(linter, condition, output) {
   if (condition) {
     warn(`Linter ${linter} unexpectedly failed. Maybe changed it's output.`);
@@ -40,6 +42,8 @@ function assert(linter, condition, output) {
  * @param {string} output Command output.
  */
 function unsuccessful(linter, exception, output) {
+  hasError = true;
+
   if (enableDebug) {
     console.log(`${linter} exception:`, exception);
   }
@@ -80,7 +84,7 @@ function header(linter) {
 }
 
 function footer() {
-  console.log("");
+  console.log("   ");
 }
 
 /**
@@ -204,4 +208,8 @@ async function eslint() {
   await editorconfigChecker();
   await cspell();
   await eslint();
+
+  if (hasError) {
+    process.exit(1);
+  }
 })();

scripts/lint/utils.cjs

@@ -8,6 +8,7 @@
 "use strict";
 
 const path = require("node:path");
+const { fileURLToPath } = require("node:url");
 
 /**
  * ANSI colors.
@@ -116,7 +117,9 @@ exports.pluralize = function (word, count) {
  * @returns Relative file path.
  */
 exports.toRelativePath = function (filePath, root) {
-  const uri = filePath.replace("file:///", "");
+  const uri = filePath.startsWith("file:///")
+    ? fileURLToPath(filePath)
+    : filePath;
   const rel = path.relative(root, uri);
   if (rel.startsWith("..")) return uri;
   return "." + path.sep + rel;