Rename and prune rules files

thyeggman · thyeggman · commit 4751f02f93f4 · 2022-10-11T16:51:54.000-07:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -18,13 +18,10 @@
 	"root": true,
 	"rules": {
 		"new-parens": "error",
-		"no-async-promise-executor": "off",
-		"no-console": "off",
 		"no-constant-condition": ["warn", { "checkLoops": false }],
+		"no-control-regex": "off",
 		"no-caller": "error",
-		"no-case-declarations": "off",
 		"no-debugger": "warn",
-		"no-dupe-class-members": "off",
 		"no-duplicate-imports": "error",
 		"no-empty": ["warn", { "allowEmptyCatch": true }],
 		"no-eval": "error",
@@ -37,18 +34,14 @@
 		"no-implied-eval": "error",
 		"no-inner-declarations": "error",
 		"no-lone-blocks": "error",
-		"no-lonely-if": "off",
 		"no-loop-func": "error",
-		"no-multi-spaces": "off",
-		"no-prototype-builtins": "off",
 		"no-return-assign": "error",
 		"no-self-compare": "error",
 		"no-sequences": "error",
 		"no-template-curly-in-string": "warn",
 		"no-throw-literal": "error",
 		"no-unmodified-loop-condition": "warn",
 		"no-unneeded-ternary": "error",
-		"no-use-before-define": "off",
 		"no-useless-call": "error",
 		"no-useless-catch": "error",
 		"no-useless-computed-key": "error",
@@ -58,28 +51,17 @@
 		"no-useless-return": "off",
 		"no-var": "error",
 		"no-with": "error",
-		"object-shorthand": "off",
 		"one-var": ["error", "never"],
-		"prefer-const": "off",
 		"prefer-numeric-literals": "error",
 		"prefer-object-spread": "error",
 		"prefer-rest-params": "error",
 		"prefer-spread": "error",
 		"quotes": ["error", "double", { "avoidEscape": true, "allowTemplateLiterals": true }],
-		"require-atomic-updates": "off",
 		"semi": ["error", "always"],
 		"semi-style": ["error", "last"],
-		"sort-imports": [
-			"error",
-			{
-				"ignoreCase": true,
-				"ignoreDeclarationSort": true,
-				"ignoreMemberSort": false,
-				"memberSyntaxSortOrder": ["none", "all", "multiple", "single"]
-			}
-		],
 		"yoda": "error",
 		"@typescript-eslint/await-thenable": "error",
+		"@typescript-eslint/ban-ts-comment": "off",
 		"@typescript-eslint/ban-types": [
 			"error",
 			{
@@ -111,9 +93,6 @@
 						"message": "`{}` actually means \"any non-nullish value\".\n- If you want a type meaning \"any object\", you probably want `object` or `Record<string, unknown>` instead.\n- If you want a type meaning \"any value\", you probably want `unknown` instead.",
 						"fixWith": "object"
 					}
-					// "object": {
-					// 	"message": "The `object` type is currently hard to use ([see this issue](https://github.com/microsoft/TypeScript/issues/21732)).\nConsider using `Record<string, unknown>` instead, as it allows you to more easily inspect and use the keys."
-					// }
 				}
 			}
 		],
@@ -124,29 +103,14 @@
 				"objectLiteralTypeAssertions": "allow-as-parameter"
 			}
 		],
-		"@typescript-eslint/explicit-function-return-type": "off",
-		"@typescript-eslint/explicit-member-accessibility": "off",
-		"@typescript-eslint/no-empty-function": "off",
 		"@typescript-eslint/no-empty-interface": "error",
-		"@typescript-eslint/no-explicit-any": "off",
 		"@typescript-eslint/no-implied-eval": "error",
 		"@typescript-eslint/no-inferrable-types": ["warn", { "ignoreParameters": true, "ignoreProperties": true }],
 		"@typescript-eslint/no-misused-promises": ["error", { "checksConditionals": false, "checksVoidReturn": false }],
-		"@typescript-eslint/no-namespace": "off",
-		"@typescript-eslint/no-non-null-assertion": "off",
-		"@typescript-eslint/no-parameter-properties": "off",
-		"@typescript-eslint/no-this-alias": "off",
-		"@typescript-eslint/no-unnecessary-condition": "off",
 		"@typescript-eslint/no-unused-expressions": ["warn", { "allowShortCircuit": true }],
 		"@typescript-eslint/no-unused-vars": ["error", { "argsIgnorePattern": "^_" }],
-		"@typescript-eslint/no-use-before-define": "off",
-		"@typescript-eslint/prefer-regexp-exec": "off",
-		"@typescript-eslint/prefer-nullish-coalescing": "off",
-		"@typescript-eslint/prefer-optional-chain": "off",
-		"@typescript-eslint/require-await": "off",
 		"@typescript-eslint/restrict-plus-operands": "error",
-		"@typescript-eslint/restrict-template-expressions": "off",
-		"@typescript-eslint/strict-boolean-expressions": "off",
+		"@typescript-eslint/triple-slash-reference": "off",
 		"@typescript-eslint/unbound-method": "off" // Too many bugs right now: https://github.com/typescript-eslint/typescript-eslint/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+unbound-method
 	}
 }
diff --git a/.prettierrc b/.prettierrc
diff --git a/.prettierrc.json b/.prettierrc.json
@@ -0,0 +1,14 @@
+{
+  "tabWidth": 2,
+  "printWidth": 120,
+  "useTabs": false,
+  "bracketSpacing": true,
+  "bracketSameLine": true,
+  "trailingComma": "es5",
+  "overrides": [
+    {
+      "files": ".prettierrc",
+      "options": { "parser": "json" }
+    }
+  ]
+}
diff --git a/package.json b/package.json
@@ -369,7 +369,9 @@
     "watch": "webpack --watch --mode development --env esbuild --info-verbosity verbose",
     "open-in-browser": "vscode-test-web --extensionDevelopmentPath=. .",
     "lint": "eslint . --ext .ts",
-    "lint-fix": "eslint . --ext .ts --fix && prettier --config .prettierrc 'src/**/*.ts' --write"
+    "lint-fix": "eslint . --ext .ts --fix && prettier --config .prettierrc.json 'src/**/*.ts' --write",
+    "prettier": "prettier .",
+    "prettier-fix": "prettier --write ."
   },
   "devDependencies": {
     "@types/glob": "^7.1.1",
