Upgrade ESLint to 3.13

Author: robertrossmann

coding-styles/fixable.js

@@ -180,6 +180,12 @@ module.exports = {
       maxEOF: 1,
     }],
 
+    // Disallow conditional expressions that can be expressed with simpler constructs
+    // Use || instead of ternary in default assignments
+    'no-unneeded-ternary': [1, {
+      defaultAssignment: false,
+    }],
+
     // Require or disallow spacing between function identifiers and their invocations
     // This rule disallows spaces between the function name and the opening parenthesis that calls
     // it.
@@ -223,6 +229,12 @@ module.exports = {
     // possible.
     'operator-assignment': [1, 'always'],
 
+    // Operator Linebreak
+    // When a statement is too long to fit on a single line, line breaks are generally inserted next
+    // to the operators separating expressions. This rule is aimed at enforcing a particular
+    // operator line break style.
+    'operator-linebreak': [1, 'before'],
+
     // Enforce padding within blocks
     // This rule enforces consistent padding within blocks.
     'padded-blocks': [1, 'never'],

coding-styles/recommended.js

@@ -137,12 +137,6 @@ module.exports = {
     // Disallow Dangling Underscores in Identifiers
     'no-underscore-dangle': 1,
 
-    // Disallow conditional expressions that can be expressed with simpler constructs
-    // Use || instead of ternary in default assignments
-    'no-unneeded-ternary': [1, {
-      defaultAssignment: false,
-    }],
-
     // Disallow Warning Comments
     // These comments are a warning signal, that there is something not production ready in your
     // code. Most likely you want to fix it or remove the comments before you roll out your code
@@ -160,12 +154,6 @@ module.exports = {
       const: 'never',
     }],
 
-    // Operator Linebreak
-    // When a statement is too long to fit on a single line, line breaks are generally inserted next
-    // to the operators separating expressions. This rule is aimed at enforcing a particular
-    // operator line break style.
-    'operator-linebreak': [1, 'before'],
-
     // Enforces having an empty line after the last top-level import statement or require call
     'import/newline-after-import': 1,
   },

package.json

@@ -12,7 +12,7 @@
     "eslint-plugin-react": "^6.8.0"
   },
   "devDependencies": {
-    "eslint": "^3.12.0"
+    "eslint": "^3.13.0"
   },
   "engines": {},
   "homepage": "https://github.com/strvcom/eslint-config-javascript",
@@ -23,7 +23,7 @@
   "license": "BSD-3-Clause",
   "main": "default",
   "peerDependencies": {
-    "eslint": "^3.12.0"
+    "eslint": "^3.13.0"
   },
   "repository": {
     "type": "git",

unused.js

@@ -119,6 +119,10 @@ module.exports = {
     // Unused, too restrictive.
     'object-curly-newline': 0,
 
+    // Prefer destructuring from arrays and objects
+    // Unused, too restrictive.
+    'prefer-destructuring': 0,
+
     // Suggest using Reflect methods where applicable
     // Deprecated, will be removed from ESLint soon
     'prefer-reflect': 0,