eslint-config-airbnb-base - npm Package Compare versions

Comparing version 14.2.0 to 14.2.1

CHANGELOG.md

@@ -0,1 +1,9 @@
+14.2.1 / 2020-11-06
+==================
+ - [base] `no-restricted-globals`: add better messages (#2320)
+ - [base] add new core eslint rules, set to off
+ - [deps] update `confusing-browser-globals`, `object.assign`
+ - [deps] update `eslint-plugin-import`, use valid `import/no-cycle` `maxDepth` option (#2250, #2249)
+ - [dev deps] update `@babel/runtime`, `eslint-find-rules`, `eslint-plugin-import`
+
 14.2.0 / 2020-06-10
@@ -2,0 +10,0 @@ ==================

package.json

 {
   "name": "eslint-config-airbnb-base",
-  "version": "14.2.0",
+  "version": "14.2.1",
   "description": "Airbnb's base JS ESLint config, following our styleguide",
@@ -56,3 +56,3 @@ "main": "index.js",
   "devDependencies": {
-    "@babel/runtime": "^7.8.7",
+    "@babel/runtime": "^7.12.5",
     "babel-preset-airbnb": "^4.5.0",
@@ -62,4 +62,4 @@ "babel-tape-runner": "^3.0.0",
     "eslint": "^5.16.0 || ^6.8.0 || ^7.2.0",
-    "eslint-find-rules": "^3.5.0",
-    "eslint-plugin-import": "^2.21.2",
+    "eslint-find-rules": "^3.6.1",
+    "eslint-plugin-import": "^2.22.1",
     "in-publish": "^2.0.1",
@@ -71,3 +71,3 @@ "safe-publish-latest": "^1.1.4",
     "eslint": "^5.16.0 || ^6.8.0 || ^7.2.0",
-    "eslint-plugin-import": "^2.21.2"
+    "eslint-plugin-import": "^2.22.1"
   },
@@ -78,6 +78,6 @@ "engines": {
   "dependencies": {
-    "confusing-browser-globals": "^1.0.9",
-    "object.assign": "^4.1.0",
+    "confusing-browser-globals": "^1.0.10",
+    "object.assign": "^4.1.2",
     "object.entries": "^1.1.2"
   }
 }

rules/errors.js

@@ -106,2 +106,7 @@ module.exports = {
 
+    // Disallow returning values from Promise executor functions
+    // https://eslint.org/docs/rules/no-promise-executor-return
+    // TODO: enable, semver-minor, once eslint v7.3 is required (which is major)
+    'no-promise-executor-return': 'off',
+
     // disallow use of Object.prototypes builtins directly
@@ -133,2 +138,9 @@ // https://eslint.org/docs/rules/no-prototype-builtins
 
+    // Disallow loops with a body that allows only one iteration
+    // https://eslint.org/docs/rules/no-unreachable-loop
+    // TODO: enable, semver-minor, once eslint v7.3 is required (which is major)
+    'no-unreachable-loop': ['off', {
+      ignore: [], // WhileStatement, DoWhileStatement, ForStatement, ForInStatement, ForOfStatement
+    }],
+
     // disallow return/throw/break/continue inside finally blocks
@@ -135,0 +147,0 @@ // https://eslint.org/docs/rules/no-unsafe-finally

rules/imports.js

@@ -236,3 +236,3 @@ module.exports = {
     // https://github.com/benmosher/eslint-plugin-import/blob/d81f48a2506182738409805f5272eff4d77c9348/docs/rules/no-cycle.md
-    'import/no-cycle': ['error', { maxDepth: Infinity }],
+    'import/no-cycle': ['error', { maxDepth: '∞' }],
 
@@ -239,0 +239,0 @@ // Ensures that there are no useless path segments

rules/style.js

@@ -110,4 +110,9 @@ module.exports = {
     // https://eslint.org/docs/rules/id-blacklist
+    // TODO: semver-major, remove once eslint v7.4+ is required
     'id-blacklist': 'off',
 
+    // disallow specified identifiers
+    // https://eslint.org/docs/rules/id-denylist
+    'id-denylist': 'off',
+
     // this option enforces minimum and maximum identifier lengths
@@ -114,0 +119,0 @@ // (variable names, property names etc.)

rules/variables.js

@@ -19,3 +19,15 @@ const confusingBrowserGlobals = require('confusing-browser-globals');
     // disallow specific globals
-    'no-restricted-globals': ['error', 'isFinite', 'isNaN'].concat(confusingBrowserGlobals),
+    '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),
 
@@ -22,0 +34,0 @@ // disallow declaration of variables already declared in the outer scope

No alert changes

Improved metrics

Total package byte prevSize

86364

increased by1.75%

Lines of code

1471

increased by1.8%

Dependency changes

• Updatedconfusing-browser-globals@^1.0.10

• Updatedobject.assign@^4.1.2