Add new rules from eslint 2.5

Author: robertrossmann

coding-styles/base.js

@@ -168,6 +168,12 @@ module.exports = {
     // This rule is aimed at increasing code clarity by discouraging deeply nesting callbacks.
     'max-nested-callbacks': [1, 4],
 
+    // Specify the Maximum Number of Statements Allowed per Line
+    // A line of code containing too many statements can be difficult to read. Code is generally
+    // read from the top down, especially when scanning, so limiting the number of statements
+    // allowed on a single line can be very beneficial for readability and maintainability.
+    'max-statements-per-line': 1,
+
     // Require Constructors to Use Initial Caps
     // This rule is aimed at helping to distinguish regular functions from constructor functions. As
     // such, it warns whenever it sees new followed by an identifier that isn't capitalized or

package.json

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

standard/best-practices.js

@@ -92,6 +92,14 @@ module.exports = {
     // loop without filtering the results in the loop.
     'guard-for-in': 1,
 
+    // Disallow duplicate exports/imports
+    // An ES6/ES2015 import can be spread over multiple lines, but this takes up unneeded
+    // whitespace. This rules validates that all imports from a single module exists in a single
+    // import statement.
+    'no-duplicate-imports': [1, {
+      includeExports: true
+    }],
+
     // Disallow eval()
     // This rule is aimed at preventing potentially dangerous, unnecessary, and slow code by
     // disallowing the use of the eval() function.

standard/known-errors.js

@@ -144,6 +144,11 @@ module.exports = {
     // class.
     'no-useless-constructor': 2,
 
+    // Disallow unnecessary escape usage
+    // Escaping non-special characters in strings and regular expressions doesn’t have any effects
+    // on results. This rule flags escapes that can be safely removed without changing behavior.
+    'no-useless-escape': 2,
+
     // Suggest using the rest parameters instead of `arguments`
     // This rule is aimed to flag usage of `arguments` variables.
     'prefer-rest-params': 2,