[Breaking] drop node < 4 support.

Also, require template literals instead of concatenation, and auto fix.

Author: ljharb

.eslintrc

@@ -2,13 +2,12 @@
     "env": {
         "node": true
     },
+    parserOptions: {
+      ecmaVersion: 2015
+    },
     ecmaFeatures: {
         jsx: true
     },
-    "globals": {
-    },
-    "plugins": [
-    ],
     "rules": {
         // Possible Errors
         "comma-dangle": [2, "never"],
@@ -149,10 +148,16 @@
         "wrap-regex": 0,
         // Legacy
         "max-depth": 0,
-        "max-len": [2, 120],
+        "max-len": [2, 120, {
+          "ignoreStrings": true,
+          "ignoreTemplateLiterals": true,
+          "ignoreComments": true,
+        }],
         "max-params": 0,
         "max-statements": 0,
         "no-plusplus": 0,
-        "no-prototype-builtins": 2
+        "no-prototype-builtins": 2,
+        "prefer-template": 2,
+        "template-curly-spacing": [2, "never"]
     }
 }

.travis.yml

@@ -4,11 +4,7 @@ node_js:
   - '6'
   - '5'
   - '4'
-  - 'iojs'
-  - '0.12'
-  - '0.10'
 before_script:
-  - 'if [ "${TRAVIS_NODE_VERSION}" = "iojs" ] || [ "${TRAVIS_NODE_VERSION}" = "0.12" ] || [ "${TRAVIS_NODE_VERSION}" = "0.10" ]; then npm install eslint@2 babel-eslint@6; fi'
 after_success:
   - npm run coveralls
 matrix:

docs/rules/forbid-elements.md

@@ -16,12 +16,12 @@ This rule checks all JSX elements and `React.createElement` calls and verifies t
 
 ### `forbid`
 
-An array of strings and/or objects. An object in this array may have the following properties: 
+An array of strings and/or objects. An object in this array may have the following properties:
 
 * `element` (required): the name of the forbidden element (e.g. `'button'`, `'Modal'`)
 * `message`: additional message that gets reported
 
-A string item in the array is a shorthand for `{ element: string }`. 
+A string item in the array is a shorthand for `{ element: string }`.
 
 The following patterns are not considered warnings:
 
@@ -50,7 +50,7 @@ React.createElement(Namespaced.Element);
 
 // [1, { "forbid": [{ "element": "button", "message": "use <Button> instead" }, "input"] }]
 <div><button /><input /></div>
-React.createElement('div', {}, React.createElemet('button', {}, React.createElement('input')));
+React.createElement('div', {}, React.createElement('button', {}, React.createElement('input')));
 ```
 
 ## When not to use

index.js

@@ -83,7 +83,7 @@ function configureAsError(rules) {
     if (!has(rules, key)) {
       continue;
     }
-    result['react/' + key] = 2;
+    result[`react/${key}`] = 2;
   }
   return result;
 }

lib/rules/forbid-component-props.js

@@ -61,7 +61,7 @@ module.exports = {
 
         context.report({
           node: node,
-          message: 'Prop `' + prop + '` is forbidden on Components'
+          message: `Prop \`${prop}\` is forbidden on Components`
         });
       }
     };

lib/rules/forbid-elements.js

@@ -59,11 +59,11 @@ module.exports = {
     });
 
     function errorMessageForElement(name) {
-      var message = '<' + name + '> is forbidden';
+      var message = `<${name}> is forbidden`;
       var additionalMessage = indexedForbidConfigs[name].message;
 
       if (additionalMessage) {
-        message = message + ', ' + additionalMessage;
+        return `${message}, ${additionalMessage}`;
       }
 
       return message;

lib/rules/forbid-foreign-prop-types.js

@@ -4,8 +4,6 @@
  */
 'use strict';
 
-var find = require('array.prototype.find');
-
 // ------------------------------------------------------------------------------
 // Constants
 // ------------------------------------------------------------------------------
@@ -47,7 +45,7 @@ module.exports = {
       },
 
       ObjectPattern: function(node) {
-        var propTypesNode = find(node.properties, function(property) {
+        var propTypesNode = node.properties.find(function(property) {
           return property.type === 'Property' && property.key.name === 'propTypes';
         });
 

lib/rules/forbid-prop-types.js

@@ -102,7 +102,7 @@ module.exports = {
         if (isForbidden(target)) {
           context.report({
             node: declaration,
-            message: 'Prop type `' + target + '` is forbidden'
+            message: `Prop type \`${target}\` is forbidden`
           });
         }
       });

lib/rules/jsx-closing-bracket-location.js

@@ -248,8 +248,7 @@ module.exports = {
         if (correctColumn !== null) {
           expectedNextLine = tokens.lastProp &&
             (tokens.lastProp.lastLine === tokens.closing.line);
-          data.details = ' (expected column ' + (correctColumn + 1) +
-            (expectedNextLine ? ' on the next line)' : ')');
+          data.details = ` (expected column ${correctColumn + 1}${expectedNextLine ? ' on the next line)' : ')'}`;
         }
 
         context.report({
@@ -274,7 +273,7 @@ module.exports = {
               case 'tag-aligned':
               case 'line-aligned':
                 return fixer.replaceTextRange([cachedLastAttributeEndPos, node.end],
-                  '\n' + getIndentation(tokens, expectedLocation, correctColumn) + closingTag);
+                  `\n${getIndentation(tokens, expectedLocation, correctColumn)}${closingTag}`);
               default:
                 return true;
             }

lib/rules/jsx-curly-spacing.js

@@ -83,7 +83,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'There should be no newline after \'' + token.value + '\'',
+        message: `There should be no newline after '${token.value}'`,
         fix: function(fixer) {
           var nextToken = sourceCode.getTokenAfter(token);
           return fixer.replaceTextRange([token.range[1], nextToken.range[0]], spaced ? ' ' : '');
@@ -101,7 +101,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'There should be no newline before \'' + token.value + '\'',
+        message: `There should be no newline before '${token.value}'`,
         fix: function(fixer) {
           var previousToken = sourceCode.getTokenBefore(token);
           return fixer.replaceTextRange([previousToken.range[1], token.range[0]], spaced ? ' ' : '');
@@ -119,7 +119,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'There should be no space after \'' + token.value + '\'',
+        message: `There should be no space after '${token.value}'`,
         fix: function(fixer) {
           var nextToken = sourceCode.getTokenAfter(token);
           var leadingComments = sourceCode.getNodeByRangeIndex(nextToken.range[0]).leadingComments;
@@ -139,7 +139,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'There should be no space before \'' + token.value + '\'',
+        message: `There should be no space before '${token.value}'`,
         fix: function(fixer) {
           var previousToken = sourceCode.getTokenBefore(token);
           var trailingComments = sourceCode.getNodeByRangeIndex(previousToken.range[0]).trailingComments;
@@ -159,7 +159,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'A space is required after \'' + token.value + '\'',
+        message: `A space is required after '${token.value}'`,
         fix: function(fixer) {
           return fixer.insertTextAfter(token, ' ');
         }
@@ -176,7 +176,7 @@ module.exports = {
       context.report({
         node: node,
         loc: token.loc.start,
-        message: 'A space is required before \'' + token.value + '\'',
+        message: `A space is required before '${token.value}'`,
         fix: function(fixer) {
           return fixer.insertTextBefore(token, ' ');
         }