@lumelabs/eslint-config 0.1.0

package/rules/variables.js

@@ -0,0 +1,59 @@
+const confusingBrowserGlobals = require("confusing-browser-globals")
+
+module.exports = {
+    rules: {
+        // enforce or disallow variable initializations at definition
+        "init-declarations": "off",
+
+        // disallow the catch clause parameter name being the same as a variable in the outer scope
+        "no-catch-shadow": "off",
+
+        // disallow deletion of variables
+        "no-delete-var": "error",
+
+        // disallow labels that share a name with a variable
+        // https://eslint.org/docs/rules/no-label-var
+        "no-label-var": "error",
+
+        // disallow specific globals
+        "no-restricted-globals": [
+            "error",
+            {
+                name: "isFinite",
+                message: "Use Number.isFinite instead https://github.com/airbnb/javascript#standard-library--isfinite",
+            },
+            {
+                name: "isNaN",
+                message: "Use Number.isNaN instead https://github.com/airbnb/javascript#standard-library--isnan",
+            },
+        ].concat(
+            confusingBrowserGlobals.map((g) => ({
+                name: g,
+                message: `Use window.${g} instead. https://github.com/facebook/create-react-app/blob/HEAD/packages/confusing-browser-globals/README.md`,
+            }))
+        ),
+
+        // disallow declaration of variables already declared in the outer scope
+        "no-shadow": "off", // Off because handled in @typescript-eslint/no-shadow
+
+        // disallow shadowing of names such as arguments
+        "no-shadow-restricted-names": "error",
+
+        // disallow use of undeclared variables unless mentioned in a /*global */ block
+        "no-undef": "error",
+
+        // disallow use of undefined when initializing variables
+        "no-undef-init": "error",
+
+        // disallow use of undefined variable
+        // https://eslint.org/docs/rules/no-undefined
+        // TODO: enable?
+        "no-undefined": "off",
+
+        // disallow declaration of variables that are not used in the code
+        "no-unused-vars": "off", // Off because handled in @typescript-eslint/no-unused-vars
+
+        // disallow use of variables before they are defined
+        "no-use-before-define": "off", // Off because handled in @typescript-eslint/no-use-before-define
+    },
+}