From e9aec9e94f7c94bdfeb3cfb419310e4eb3d36593 Mon Sep 17 00:00:00 2001
From: Jordan Harband <ljharb@gmail.com>
Date: Fri, 6 May 2016 12:24:13 -0700
Subject: [PATCH] [eslint config] [base] update `eslint-plugin-import`,
 `eslint-find-rules`

---
 packages/eslint-config-airbnb-base/package.json |  6 +++---
 packages/eslint-config-airbnb-base/rules/es6.js | 15 ++++++++++++++-
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/packages/eslint-config-airbnb-base/package.json b/packages/eslint-config-airbnb-base/package.json
index bf860c2c..ea064e7e 100644
--- a/packages/eslint-config-airbnb-base/package.json
+++ b/packages/eslint-config-airbnb-base/package.json
@@ -45,12 +45,12 @@
   "devDependencies": {
     "babel-tape-runner": "^1.3.1",
     "eslint": "^2.9.0",
-    "eslint-find-rules": "^1.8.0",
-    "eslint-plugin-import": "^1.6.1",
+    "eslint-find-rules": "^1.9.2",
+    "eslint-plugin-import": "^1.7.0",
     "tape": "^4.5.1"
   },
   "peerDependencies": {
     "eslint": "^2.9.0",
-    "eslint-plugin-import": "^1.6.1"
+    "eslint-plugin-import": "^1.7.0"
   }
 }
diff --git a/packages/eslint-config-airbnb-base/rules/es6.js b/packages/eslint-config-airbnb-base/rules/es6.js
index fc85a783..aa6e64fd 100644
--- a/packages/eslint-config-airbnb-base/rules/es6.js
+++ b/packages/eslint-config-airbnb-base/rules/es6.js
@@ -29,6 +29,7 @@ module.exports = {
     // enforce the spacing around the * in generator functions
     'generator-star-spacing': 0,
     // disallow modifying variables of class declarations
+    // TODO: enable
     'no-class-assign': 0,
     // disallow arrow functions where they could be confused with comparisons
     // http://eslint.org/docs/rules/no-confusing-arrow
@@ -110,7 +111,10 @@ module.exports = {
     // Forbid the use of extraneous packages
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-extraneous-dependencies.md
     // 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
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-unresolved.md
     'import/no-unresolved': [2, { 'commonjs': true }],
@@ -152,7 +156,16 @@ module.exports = {
     // TODO: enable?
     'import/order': [0, {
       '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
+    // TODO: enable
+    'import/no-mutable-exports': 0,
   },
   'settings': {
     'import/resolver': {