chore: ts-check and lint eslintrc

Config as code for a reason, eh?

Author: lishaduck

.eslintrc.js

@@ -1,3 +1,4 @@
+// @ts-check
 // TODO(@lishaduck) [eslint@>9.5]: Use `.ts` extension to get more type checking for this file.
 // TODO(@lishaduck) [engine:node@>=18]: Upgrade `tseslint`.
 // TODO(@lishaduck) [engine:node@>=18]: Use `eslint-define-config` to get type checking for this file.
@@ -32,7 +33,6 @@ module.exports = {
     'test/snapshots',
     'test/temporary',
     'vendor/',
-    '.eslintrc.js',
     'new-package/elm-review-package-tests/check-previews-compile.js'
   ],
   rules: {
@@ -114,6 +114,13 @@ module.exports = {
         'n/no-process-exit': 'off',
         '@typescript-eslint/unbound-method': 'off' // TODO(@lishaduck): Fix this warning. I just got confused.
       }
+    },
+    {
+      files: ['.eslintrc.js'],
+      rules: {
+        camelcase: 'off',
+        '@typescript-eslint/prefer-nullish-coalescing': 'off' // It's not happy with how TS is set up on this config file.
+      }
     }
   ],
   globals: {

package.json

@@ -47,7 +47,7 @@
     "check-engines": "ls-engines --mode=ideal",
     "elm-format": "elm-format --validate ast-codec init-templates new-package parseElm review template",
     "elm-tests": "(cd template/ && elm make src/Elm/Review/Main.elm --output=/dev/null && elm-test)",
-    "eslint-check": "eslint . --report-unused-disable-directives --max-warnings=2",
+    "eslint-check": "eslint . --report-unused-disable-directives --max-warnings=7",
     "eslint-fix": "npm run eslint-check -- --fix",
     "jest": "npx jest",
     "prepare": "elm-tooling install",

tsconfig.json

@@ -31,7 +31,6 @@
   "exclude": [
     "node_modules",
     "**/elm-stuff/",
-    ".eslintrc.js",
     "new-package/elm-review-package-tests/check-previews-compile.js",
     "test/temporary"
   ]