Merge pull request #27 from AlCalzone/dependabot/npm_and_yarn/typescr…

…ipt-4.0.2

build(deps-dev): bump typescript from 3.9.7 to 4.0.2

.eslintrc.js

@@ -11,6 +11,7 @@ module.exports = {
 		"plugin:prettier/recommended", // Enables eslint-plugin-prettier and displays prettier errors as ESLint errors. Make sure this is always the last configuration in the extends array.
 	],
 	plugins: [],
+	reportUnusedDisableDirectives: true,
 	rules: {
 		// Place to specify ESLint rules. Can be used to overwrite rules specified from the extended configs
 		"@typescript-eslint/no-parameter-properties": "off",
@@ -30,32 +31,73 @@ module.exports = {
 				argsIgnorePattern: "^_",
 			},
 		],
-		"@typescript-eslint/explicit-function-return-type": [
-			"warn",
-			{
-				allowExpressions: true,
-				allowTypedFunctionExpressions: true,
-			},
-		],
 		"@typescript-eslint/no-object-literal-type-assertion": "off",
 		"@typescript-eslint/interface-name-prefix": "off",
 		"@typescript-eslint/no-non-null-assertion": "off", // This is necessary for Map.has()/get()!
 		"@typescript-eslint/no-inferrable-types": [
 			"error",
 			{
-				"ignoreProperties": true,
-				"ignoreParameters": true
-			}
+				ignoreProperties: true,
+				ignoreParameters: true,
+			},
+		],
+		"@typescript-eslint/ban-ts-comment": [
+			"error",
+			{
+				"ts-expect-error": false,
+				"ts-ignore": true,
+				"ts-nocheck": true,
+				"ts-check": false,
+			},
+		],
+		"@typescript-eslint/restrict-template-expressions": [
+			"error",
+			{
+				allowNumber: true,
+				allowBoolean: true,
+				// This is necessary to log errors
+				// TODO: Consider switching to false when we may annotate catch clauses
+				allowAny: true,
+				allowNullish: true,
+			},
+		],
+		"@typescript-eslint/no-misused-promises": [
+			"error",
+			{
+				checksVoidReturn: false,
+			},
+		],
+		// We can turn this on from time to time but in general these rules
+		// make our lives harder instead of easier
+		"@typescript-eslint/no-unsafe-assignment": "off",
+		"@typescript-eslint/no-unsafe-member-access": "off",
+		"@typescript-eslint/no-unsafe-return": "off",
+		"@typescript-eslint/no-unsafe-call": "off",
+
+		// Although this rule makes sense, it takes about a second to execute (and we don't need it)
+		"@typescript-eslint/no-implied-eval": "off",
+
+		"@typescript-eslint/explicit-module-boundary-types": [
+			"warn",
+			{ allowArgumentsExplicitlyTypedAsAny: true },
 		],
-		"@typescript-eslint/ban-ts-ignore": "warn",
+		"@typescript-eslint/no-this-alias": "off",
 	},
 	overrides: [
 		{
 			files: ["*.test.ts"],
 			rules: {
 				"@typescript-eslint/explicit-function-return-type": "off",
 				"@typescript-eslint/no-empty-function": "off",
-				"@typescript-eslint/ban-ts-ignore": "off",
+				"@typescript-eslint/ban-ts-comment": "off",
+				"@typescript-eslint/no-unsafe-assignment": "off",
+				"@typescript-eslint/no-unsafe-member-access": "off",
+				"@typescript-eslint/no-unsafe-member-return": "off",
+				"@typescript-eslint/no-unsafe-return": "off",
+				"@typescript-eslint/no-unsafe-call": "off",
+				"@typescript-eslint/no-floating-promises": "off",
+				"@typescript-eslint/require-await": "off",
+				"@typescript-eslint/unbound-method": "warn",
 			},
 		},
 		{