@html-validate/eslint-config 5.6.0 → 5.7.1

package/eslint.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 
 const path = require("path");
-const { spawn } = require("child_process"); // eslint-disable-line security/detect-child-process
+const { spawn } = require("child_process");
 
 const pkgPath = path.dirname(require.resolve("eslint/package.json"));
 const binary = path.join(pkgPath, "bin/eslint");
@@ -9,6 +9,5 @@ const binary = path.join(pkgPath, "bin/eslint");
 spawn("node", [binary, ...process.argv.slice(2)], {
 	stdio: "inherit",
 }).on("exit", (code) => {
-	/* eslint-disable-next-line no-process-exit */
-	process.exit(code);
+	process.exitCode = code;
 });

package/index.js

@@ -17,6 +17,7 @@ module.exports = {
 
 	extends: [
 		require.resolve("eslint-config-sidvind/es2017"),
+		"plugin:eslint-comments/recommended",
 		"plugin:prettier/recommended",
 		"plugin:import/errors",
 		"plugin:node/recommended-module",
@@ -28,6 +29,13 @@ module.exports = {
 	plugins: ["array-func", "prettier", "import", "node", "security", "sonarjs"],
 
 	rules: {
+		"eslint-comments/disable-enable-pair": ["error", { allowWholeFile: true }],
+		"eslint-comments/require-description": [
+			"error",
+			{ ignore: ["eslint-enable", "eslint-env", "exported", "global", "globals"] },
+		],
+		"eslint-comments/no-unused-disable": "error",
+
 		"import/default": "off",
 		"import/extensions": ["error", "never", { json: "always" }],
 		"import/newline-after-import": "error",
@@ -47,10 +55,11 @@ module.exports = {
 		 * work with typescript */
 		"node/no-missing-import": "off",
 
+		"security/detect-child-process": "off", // produces more noise than useful errors
 		"security/detect-non-literal-fs-filename": "off", // html-validate reads files, don't want to acknowledge all occurrences
 		"security/detect-non-literal-regexp": "error",
 		"security/detect-non-literal-require": "error",
-		"security/detect-object-injection": "error",
+		"security/detect-object-injection": "off", // produces more noise than useful errors
 		"security/detect-unsafe-regex": "error",
 
 		"sonarjs/no-small-switch": "off", // prefer to use small switches when the intention is to all more cases later

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@html-validate/eslint-config",
-  "version": "5.6.0",
+  "version": "5.7.1",
   "description": "Eslint sharable config used by the various HTML-validate packages",
   "keywords": [
     "eslint"
@@ -41,6 +41,7 @@
     "eslint-import-resolver-node": "0.3.7",
     "eslint-import-resolver-typescript": "3.5.3",
     "eslint-plugin-array-func": "3.1.8",
+    "eslint-plugin-eslint-comments": "3.2.0",
     "eslint-plugin-import": "2.27.5",
     "eslint-plugin-node": "11.1.0",
     "eslint-plugin-prettier": "4.2.1",
@@ -62,5 +63,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "887d53ec6473b2d4a4607cc6eea1e9a06c8cd9b0"
+  "gitHead": "66326d4a5ecfd939be5bb70e0f386c1fc2d8fcc4"
 }