(breaking) Add support for Node.js v6

Author: robertrossmann

default.js

@@ -2,7 +2,7 @@
 
 module.exports = {
   extends: [
-    './environments/nodejs/latest.js',
+    './environments/nodejs/v6.js',
     './environments/nodejs/best-practices.js',
     './environments/nodejs/optional.js'
   ]

environments/nodejs/README.md

@@ -4,19 +4,19 @@ These configuration files are suitable to lint code which will run on Node.js.
 
 ## Configurations
 
-### javascript/environments/nodejs/latest
+### @strv/javascript/environments/nodejs/v6
 
-Suitable for the very latest version of Node.js. It configures ESLint to allow known ES 2015 features and disallow those still missing as well as includes Node-specific ruleset for known errors.
+Suitable for projects running on Node.js v6. Also includes Node-specific ruleset for known errors.
 
-### javascript/environments/nodejs/legacy
+### @strv/javascript/environments/nodejs/v5
 
-Suitable for legacy versions of Node.js (0.12 and older). Also includes Node-specific rulesets for known errors.
+Suitable for projects running on Node.js v5. It configures ESLint to allow known ES 2015 features and disallow those still missing. Also includes Node-specific rulesets for known errors.
 
-### javascript/environments/nodejs/best-practices
+### @strv/javascript/environments/nodejs/best-practices
 
 Use this ruleset in conjunction with one of the version-specific configuration files. Provides additional safety checks to flag potentially unsafe code.
 
-### javascript/environments/nodejs/optional
+### @strv/javascript/environments/nodejs/optional
 
 Use this ruleset in conjunction with any of the above. Provides additional insights into potential inconsistencies in the project.
 

environments/nodejs/best-practices.js

@@ -41,7 +41,8 @@ module.exports = {
     'no-sync': 1,
 
     // Suggest using Reflect methods where applicable
-    // @TODO: Requires Node.js support
+    // This rule should be enabled per environment because not all environments might have support
+    // for the Reflect API.
     'prefer-reflect': 0,
 
     // Disallow process.env

environments/nodejs/known-errors.js

@@ -32,8 +32,7 @@ module.exports = {
     // trace.
     'no-process-exit': 2,
 
-    // Suggest using the rest parameters instead of `arguments`
-    // @TODO: Requires Node.js support
-    'prefer-rest-params': 0
+    // Enforce usage of the rest parameters instead of the weird `arguments`
+    'prefer-rest-params': 2
   }
 }

environments/nodejs/legacy.js

@@ -8,8 +8,10 @@
 
 'use strict'
 
+/**
+ * This file configures ESLint to work with the Node.js 5.0 release
+ */
 module.exports = {
-  // This file configures ESLint to work with the newest ES 2015 syntax and features
   extends: './known-errors.js',
 
   env: {
@@ -22,8 +24,13 @@ module.exports = {
   },
 
   rules: {
-    // ESLint no longer supports disabling some still unsuported ES 2015 features, so we need to use
-    // this rule to disable that syntax
+    // Node.js v5 does not support Reflect API
+    'prefer-reflect': 0,
+
+    // Node.js v5 does not support rest params
+    'prefer-rest-params': 0,
+
+    // The following ES 2015 syntax is not supported by Node.js v5
     'no-restricted-syntax': [
       2,
       // Rest parameters (not to be confused with rest arguments)
@@ -32,7 +39,7 @@ module.exports = {
       'AssignmentPattern',
       // Destructuring assignment
       'ObjectPattern',
-      // Experimental object/rest spread
+      // Experimental object/rest spread (not in ES 2015)
       'ExperimentalRestProperty'
     ]
   }

environments/nodejs/latest.js renamed to environments/nodejs/v5.js

@@ -0,0 +1,38 @@
+/**
+ * js-coding-standards
+ *
+ * @author      Robert Rossmann <[email protected]>
+ * @copyright   2016 STRV
+ * @license     http://choosealicense.com/licenses/bsd-3-clause  BSD-3-Clause License
+ */
+
+'use strict'
+
+/**
+ * This file configures ESLint to work with the Node.js 6.0 release
+ */
+module.exports = {
+  extends: './known-errors.js',
+
+  env: {
+    node: true,
+    es6: true
+  },
+
+  parserOptions: {
+    ecmaVersion: 6
+  },
+
+  rules: {
+    // ESLint no longer supports disabling some still unsuported ES 2015 features, so we need to use
+    // this rule to disable that syntax
+    'no-restricted-syntax': [
+      2,
+      // Experimental object/rest spread (not in ES 2015)
+      'ExperimentalRestProperty'
+    ],
+
+    // Suggest using Reflect API where applicable
+    'prefer-reflect': 1
+  }
+}

environments/nodejs/v6.js

@@ -16,7 +16,7 @@ module.exports = {
     // overwriting some configuration options. Here, we are including the
     // base configuration for Node.js development and we opt-in to the
     // best-practices ruleset.
-    '@strv/javascript/environments/nodejs/latest',
+    '@strv/javascript/environments/nodejs/v6',
     '@strv/javascript/environments/nodejs/best-practices',
     // If you are brave enough, you can also extend the "optional" ruleset
     '@strv/javascript/environments/nodejs/optional'