Create per-project lint configurations

This allows for linting a single focused workspace instead of all workspaces.

Isolate project-specific ESLint configs in their respective config files.

Author: wimpyprogrammer

.eslintrc.json

@@ -1,8 +1,8 @@
 {
+	"root": true,
 	"parser": "@typescript-eslint/parser",
 	"extends": [
 		"airbnb-base",
-		"plugin:jest/recommended",
 		"plugin:@typescript-eslint/eslint-recommended",
 		"plugin:@typescript-eslint/recommended",
 		"prettier"
@@ -11,47 +11,25 @@
 		"ecmaVersion": 2018,
 		"project": ["./**/tsconfig.json", "./**/tsconfig.test.json"]
 	},
+	"ignorePatterns": ["coverage/", "lib/"],
 	"rules": {
 		"import/extensions": [
 			"error",
 			{
 				"ts": "never"
 			}
 		],
-		"import/no-cycle": "off",
-		"import/no-extraneous-dependencies": [
-			"error",
-			{
-				"devDependencies": ["**/*.spec.{js,ts}", "**/*.config.js"]
-			}
-		],
-		"import/no-unresolved": [
-			"error",
-			{
-				"ignore": [
-					"history",
-					"lodash",
-					// Workaround for alexgorbatchev/eslint-import-resolver-typescript#17
-					"regexp-tree/ast",
-					"RegexColorizer"
-				]
-			}
-		],
 		"import/prefer-default-export": "off",
-		"no-else-return": "off",
 		"no-plusplus": "off",
-		// This rule excludes too many escape sequences we need, like \p
-		"no-useless-escape": "off",
 		// Allow ForOfStatement, preserve the rest of Airbnb's restrictions
 		"no-restricted-syntax": [
 			"error",
 			"ForInStatement",
 			"LabeledStatement",
 			"WithStatement"
-		],
-		"@typescript-eslint/explicit-function-return-type": "off",
-		"@typescript-eslint/explicit-module-boundary-types": "off"
+		]
 	},
+	"reportUnusedDisableDirectives": true,
 	"settings": {
 		"import/parsers": {
 			"@typescript-eslint/parser": [".ts"]

.github/workflows/tests.yml

@@ -50,7 +50,7 @@ jobs:
       - run: npx prettier --check .
         working-directory: ./
       - run: yarn run build
-      - run: yarn run lint
+      - run: yarn run -T lint
       - run: yarn run test
 
       - run: npx testpack-cli --keep=@types/*,ts-jest,typescript jest.config.js tsconfig.test.json src/e2e.spec.ts

demo/.eslintrc.json

@@ -0,0 +1,16 @@
+{
+	"rules": {
+		"import/no-extraneous-dependencies": [
+			"error",
+			{
+				"devDependencies": ["**/*.config.js"]
+			}
+		],
+		"import/no-unresolved": [
+			"error",
+			{
+				"ignore": ["history", "lodash", "RegexColorizer"]
+			}
+		]
+	}
+}

demo/package.json

@@ -4,10 +4,12 @@
   "private": true,
   "scripts": {
     "build": "wp --config webpack.config.js",
+    "lint": "tsc -b ./tsconfig.lint.json && eslint .",
     "test": "exit 0"
   },
   "devDependencies": {
     "@types/lodash": "^4.14.140",
+    "@types/node": "^24.8.1",
     "clean-webpack-plugin": "^4.0.0",
     "css-loader": "^7.1.1",
     "node-sass": "^9.0.0",

tsconfig.lint.json renamed to demo/tsconfig.lint.json

@@ -2,6 +2,6 @@
 	"compilerOptions": {
 		"noEmit": true
 	},
-	"extends": "./tsconfig.json",
+	"extends": "../tsconfig.json",
 	"references": [{ "path": "./" }]
 }

package.json

@@ -8,7 +8,7 @@
   "scripts": {
     "build": "yarn workspaces run build",
     "format": "prettier --write .",
-    "lint": "tsc -b ./tsconfig.lint.json && eslint . --report-unused-disable-directives",
+    "lint": "yarn workspaces foreach --all run lint",
     "test": "yarn workspaces run test"
   },
   "devDependencies": {
@@ -19,7 +19,7 @@
     "eslint-config-prettier": "^10.0.1",
     "eslint-import-resolver-typescript": "^4.3.1",
     "eslint-plugin-import": "^2.25.3",
-    "eslint-plugin-jest": "^27.0.4",
-    "prettier": "2.2.1"
+    "prettier": "2.2.1",
+    "typescript": "^4.2.3"
   }
 }

packages/regex-to-strings/.eslintrc.json

@@ -0,0 +1,24 @@
+{
+	"extends": ["plugin:jest/recommended"],
+	"rules": {
+		"import/no-cycle": "off",
+		"import/no-extraneous-dependencies": [
+			"error",
+			{
+				"devDependencies": ["**/*.spec.{js,ts}", "**/*.config.js"]
+			}
+		],
+		"import/no-unresolved": [
+			"error",
+			{
+				"ignore": [
+					// Workaround for alexgorbatchev/eslint-import-resolver-typescript#17
+					"regexp-tree/ast"
+				]
+			}
+		],
+		"no-else-return": "off",
+		// This rule excludes too many escape sequences we need, like \p
+		"no-useless-escape": "off"
+	}
+}

packages/regex-to-strings/package.json

@@ -29,6 +29,7 @@
   "scripts": {
     "build": "tsc -b --clean && tsc -b",
     "changelog": "npx auto-changelog",
+    "lint": "tsc -b ./tsconfig.lint.json && eslint .",
     "release": "npx release-it --no-increment",
     "test": "jest --coverage"
   },
@@ -37,11 +38,11 @@
     "@types/jest-when": "^3.5.0",
     "jest": "^29.7.0",
     "jest-when": "^3.2.1",
-    "ts-jest": "^29.2.5",
-    "typescript": "^4.2.3"
+    "ts-jest": "^29.2.5"
   },
   "dependencies": {
     "escape-string-regexp": "^4.0.0",
+    "eslint-plugin-jest": "^27.0.4",
     "pandemonium": "^2.0.0",
     "regexp-tree": "^0.1.23"
   },

packages/regex-to-strings/tsconfig.lint.json

@@ -0,0 +1,7 @@
+{
+	"compilerOptions": {
+		"noEmit": true
+	},
+	"extends": "../../tsconfig.json",
+	"references": [{ "path": "./" }]
+}