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

Comparing version 2.0.0 to 3.0.0

CHANGELOG.md

@@ -0,1 +1,11 @@
+3.0.0 / 2016-05-07
+==================
+ - [breaking] enable `import/no-mutable-exports`
+ - [breaking] enable `no-class-assign` rule, to pair with `no-func-assign`
+ - [breaking] widen `no-extra-parens` to include everything, except `nestedBinaryExpressions`
+ - [breaking] Re-enabling `newline-per-chained-call` (#748)
+ - [minor] enable `import/no-amd`
+ - [patch] enable `import/no-duplicates`
+ - [deps] update `eslint`, `eslint-plugin-import`, `eslint-find-rules`
+
 2.0.0 / 2016-04-29
@@ -2,0 +12,0 @@ ==================

package.json

 {
   "name": "eslint-config-airbnb-base",
-  "version": "2.0.0",
+  "version": "3.0.0",
   "description": "Airbnb's base JS ESLint config, following our styleguide",
@@ -48,4 +48,4 @@ "main": "index.js",
     "eslint": "^2.9.0",
-    "eslint-find-rules": "^1.7.0",
-    "eslint-plugin-import": "^1.6.1",
+    "eslint-find-rules": "^1.9.2",
+    "eslint-plugin-import": "^1.7.0",
     "tape": "^4.5.1"
@@ -55,4 +55,4 @@ },
     "eslint": "^2.9.0",
-    "eslint-plugin-import": "^1.6.1"
+    "eslint-plugin-import": "^1.7.0"
   }
 }

rules/errors.js

@@ -28,3 +28,7 @@ module.exports = {
     // disallow unnecessary parentheses
-    'no-extra-parens': [2, 'functions'],
+    // http://eslint.org/docs/rules/no-extra-parens
+    'no-extra-parens': [2, 'all', {
+      'conditionalAssign': true,
+      'nestedBinaryExpressions': false,
+    }],
     // disallow unnecessary semicolons
@@ -31,0 +35,0 @@ 'no-extra-semi': 2,

rules/es6.js

@@ -32,3 +32,4 @@ module.exports = {
     // disallow modifying variables of class declarations
-    'no-class-assign': 0,
+    // http://eslint.org/docs/rules/no-class-assign
+    'no-class-assign': 2,
     // disallow arrow functions where they could be confused with comparisons
@@ -114,3 +115,6 @@ // http://eslint.org/docs/rules/no-confusing-arrow
     // TODO: enable
-    'import/no-extraneous-dependencies': [0, { 'devDependencies': false }],
+    'import/no-extraneous-dependencies': [0, {
+      'devDependencies': false,
+      'optionalDependencies': false,
+    }],
     // ensure imports point to files/modules that can be resolved
@@ -128,4 +132,3 @@ // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-amd.md
-    // TODO: enable
-    'import/no-amd': 0,
+    'import/no-amd': 2,
     // disallow non-import statements appearing before import statements
@@ -136,5 +139,4 @@ // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/imports-first.md
     // disallow duplicate imports
-    // TODO: enable
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
-    'import/no-duplicates': 0,
+    'import/no-duplicates': 2,
     // disallow use of jsdoc-marked-deprecated imports
@@ -159,3 +161,11 @@ // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-deprecated.md
       'groups': ['builtin', 'external', 'internal', 'parent', 'sibling', 'index'],
+      'newlines-between': 'never',
     }],
+    // Require a newline after the last import/require in a group
+    // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/newline-after-import.md
+    // TODO: enable
+    'import/newline-after-import': 0,
+    // Forbid mutable exports
+    // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-mutable-exports.md
+    'import/no-mutable-exports': 2,
   },
@@ -162,0 +172,0 @@ 'settings': {

rules/style.js

@@ -73,3 +73,3 @@ module.exports = {
     // http://eslint.org/docs/rules/newline-per-chained-call
-    'newline-per-chained-call': [0, { 'ignoreChainWithDepth': 3 }],
+    'newline-per-chained-call': [2, { 'ignoreChainWithDepth': 3 }],
     // disallow use of the Array constructor
@@ -76,0 +76,0 @@ 'no-array-constructor': 2,

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

33022

increased by3.39%

Lines of code

717

increased by1.99%

No dependency changes