eslint-config-isaacscript 1.0.48 → 1.0.52

package/.cspell.json

@@ -16,6 +16,7 @@
         "sarisia",
         "technote",
         "turnbull",
+        "unexported",
         "unfixable",
         "winpty",
         "zamiell"

package/.vscode/extensions.json

@@ -3,6 +3,6 @@
 // https://go.microsoft.com/fwlink/?LinkId=827846
 {
   "recommendations": [
-    "streetsidesoftware.code-spell-checker", // A spell-checker extension based on cspell is used
+    "streetsidesoftware.code-spell-checker", // A spell-checker extension based on cspell
   ]
 }

package/base.js

@@ -34,10 +34,19 @@ module.exports = {
     // https://github.com/Zamiell/eslint-plugin-eqeqeq-fix
     "plugin:eqeqeq-fix/recommended",
 
+    // This prevents implicit iteration of Maps and Sets,
+    // which is helpful to prevent bugs when refactoring
+    // https://github.com/Zamiell/eslint-plugin-no-implicit-map-set-loops
+    "plugin:no-implicit-map-set-loops/recommended",
+
     // This provides a version of the "no-template-curly-in-string" that the "--fix" flag can fix
     // https://github.com/Zamiell/eslint-plugin-no-template-curly-in-string-fix
     "plugin:no-template-curly-in-string-fix/recommended",
 
+    // This prevents void return types on unexported functions
+    // https://github.com/Zamiell/eslint-plugin-no-void-return-type
+    "plugin:no-void-return-type/recommended",
+
     // We use Prettier to automatically format TypeScript files
     // We want to run Prettier as an ESLint rule so that we can detect non-formatted files in CI
     // https://github.com/prettier/eslint-plugin-prettier
@@ -201,7 +210,8 @@ module.exports = {
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/prefer-default-export.md
     // Defined at:
     // https://github.com/airbnb/javascript/blob/master/packages/eslint-config-airbnb-base/rules/imports.js
-    // It is common to only export one thing with the intention of adding more exports later
+    // The case against default exports is layed out here:
+    // https://basarat.gitbook.io/typescript/main-1/defaultisbad
     "import/prefer-default-export": "off",
 
     // Documentation:

package/mod.js

@@ -5,16 +5,20 @@ module.exports = {
     // The mod config extends the base configuration:
     // https://github.com/IsaacScript/eslint-config-isaacscript/blob/main/base.js
     "./base",
-
-    // This prevents implicit iteration of Maps and Sets,
-    // since it does not work robustly with the TypeScriptToLua transpiler
-    // https://github.com/Zamiell/eslint-plugin-no-implicit-map-set-loops
-    "plugin:no-implicit-map-set-loops/recommended",
   ],
 
   // We modify the linting rules from the base for some specific things
   // (listed in alphabetical order)
   rules: {
+    // Documentation:
+    // https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/docs/rules/no-loop-func.md
+    // https://eslint.org/docs/rules/no-loop-func
+    // Defined at:
+    // https://github.com/iamturns/eslint-config-airbnb-typescript/blob/master/lib/shared.js
+    // This rule throws false positives with API functions
+    // It can be worked around by supplying lists of globals to ESLint, but this is ugly
+    "@typescript-eslint/no-loop-func": "off",
+
     // Documentation:
     // https://github.com/eslint/eslint/blob/master/docs/rules/no-bitwise.md
     // Defined at:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-isaacscript",
-  "version": "1.0.48",
+  "version": "1.0.52",
   "description": "An ESLint config for IsaacScript projects.",
   "repository": {
     "type": "git",
@@ -8,14 +8,9 @@
   },
   "license": "GPL-3.0",
   "author": "Zamiell",
-  "dependencies": {
-    "eslint-config-airbnb-base": "^14.2.1",
-    "eslint-config-airbnb-typescript": "^14.0.0",
-    "eslint-config-prettier": "^8.3.0"
-  },
   "devDependencies": {
-    "eslint": "^7.32.0",
+    "eslint": "^8.3.0",
     "eslint-plugin-prettier": "^4.0.0",
-    "prettier": "^2.4.0"
+    "prettier": "^2.5.0"
   }
 }