Upgrade ESLint to 2.12, add new rules

Author: robertrossmann

coding-styles/base.js

@@ -302,6 +302,11 @@ module.exports = {
     // This rule is aimed at ensuring consistency of string quotes.
     quotes: [1, 'single', 'avoid-escape'],
 
+    // Enforce spacing between rest and spread operators and their expressions
+    // This rule aims to enforce consistent spacing between rest and spread operators and their
+    // expressions.
+    'rest-spread-spacing': [1, 'never'],
+
     // Enforce spacing before and after semicolons
     'semi-spacing': [1, {
       before: false,

package.json

@@ -17,7 +17,7 @@
   "license": "BSD-3-Clause",
   "main": "default",
   "peerDependencies": {
-    "eslint": "^2.9.0"
+    "eslint": "^2.12.0"
   },
   "repository": {
     "type": "git",
@@ -27,6 +27,6 @@
     "lint": "eslint standard environments coding-styles unused.js default.js"
   },
   "devDependencies": {
-    "eslint": "^2.9.0"
+    "eslint": "^2.12.0"
   }
 }

standard/best-practices.js

@@ -53,7 +53,15 @@ module.exports = {
     //
     // @see  https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/
     //       Operator_Precedence
-    'no-extra-parens': 1,
+    'no-extra-parens': [1, 'all', {
+      nestedBinaryExpressions: false
+    }],
+
+    // Disallow mixes of different operators
+    // Enclosing complex expressions by parentheses clarifies the developer’s intention, which makes
+    // the code more readable. This rule warns when different operators are used consecutively
+    // without parentheses in an expression.
+    'no-mixed-operators': 1,
 
     // Disallow Initializing to undefined
     // In JavaScript, a variable that is declared and not initialized to any value automatically

standard/optional.js

@@ -46,6 +46,15 @@ module.exports = {
     // code to a certain number of characters.
     'max-len': [1, 100, 2],
 
+    // Enforce a maximum file length
+    // This rule enforces a maximum number of lines per file, in order to aid in maintainability and
+    // reduce complexity.
+    'max-lines': [1, {
+      max: 600,
+      skipBlankLines: true,
+      skipComments: true
+    }],
+
     // Limit Maximum Number of Parameters
     // Functions that take numerous parameters can be difficult to read and write because it
     // requires the memorization of what each parameter is, its type, and the order they should

unused.js

@@ -104,6 +104,10 @@ module.exports = {
     // Unused, too restrictive.
     'object-property-newline': 0,
 
+    // Require or disallow line breaks inside braces
+    // Unused, too restrictive.
+    'object-curly-newline': 0,
+
     // eslint-plugin-react
 
     // Forbid certain propTypes