fix: update to node >= 6 and fix broken linting

Author: RyanTheAllmighty

package-lock.json

@@ -24,6 +24,6 @@
     "babel-preset-stage-0": "^6.24.1"
   },
   "engines": {
-    "node": ">= 4"
+    "node": ">= 6"
   }
 }

packages/babel-preset-atlauncher/package.json

@@ -41,13 +41,13 @@
     "eslint": "^3.19.0",
     "eslint-plugin-eslint-comments": "^1.0.1",
     "eslint-plugin-filenames": "^1.2.0",
-    "eslint-plugin-import": "^2.2.0",
-    "eslint-plugin-jsx-a11y": "^4.0.0",
+    "eslint-plugin-import": "^2.3.0",
+    "eslint-plugin-jsx-a11y": "^5.0.3",
     "eslint-plugin-no-empty-blocks": "^0.0.2",
     "eslint-plugin-promise": "^3.5.0",
-    "eslint-plugin-react": "^6.10.3"
+    "eslint-plugin-react": "^7.0.1"
   },
   "engines": {
-    "node": ">= 4"
+    "node": ">= 6"
   }
 }

packages/eslint-config-atlauncher/package.json

@@ -31,7 +31,7 @@ module.exports = {
         'jsx-a11y/iframe-has-title': 'error',
 
         // require all image tags to have an `alt` property
-        'jsx-a11y/img-has-alt': 'error',
+        'jsx-a11y/alt-text': 'error',
 
         // don't allow the workds `image`, `photo` or `picture` in image `alt` property
         'jsx-a11y/img-redundant-alt': 'error',
@@ -67,6 +67,12 @@ module.exports = {
         'jsx-a11y/scope': 'error',
 
         // don't allow positive values for `tabIndex`
-        'jsx-a11y/tabindex-no-positive': 'error'
+        'jsx-a11y/tabindex-no-positive': 'error',
+
+        // warn when media doesn't have caption`
+        'jsx-a11y/media-has-caption': 'warn',
+
+        // don't allow tabIndex on non interactive elements
+        'jsx-a11y/no-noninteractive-tabindex': 'error'
     }
 };

packages/eslint-config-atlauncher/rules/plugin-jsx-a11y.js

@@ -1,7 +1,7 @@
 module.exports = {
     settings: {
         react: {
-            version: '15.5.3'
+            version: '15.5.4'
         }
     },
     rules: {
@@ -157,7 +157,11 @@ module.exports = {
         'react/jsx-pascal-case': 'error',
 
         // require a space before self closing tags such as `<A prop={prop} />` rather than `<A prop={prop}/>`
-        'react/jsx-space-before-closing': 'error',
+        'react/jsx-tag-spacing': ['error', {
+            closingSlash: 'never',
+            beforeSelfClosing: 'always',
+            afterOpening: 'never'
+        }],
 
         // ensures components that import React don't show it as being unused
         'react/jsx-uses-react': 'error',
@@ -166,6 +170,9 @@ module.exports = {
         'react/jsx-uses-vars': 'error',
 
         // ensures multiline JSX is wrapped within parenthesis
-        'react/jsx-wrap-multilines': 'error'
+        'react/jsx-wrap-multilines': 'error',
+
+        // prevent usage of setState in componentWillUpdate
+        'react/no-will-update-set-state': 'error'
     }
 };

packages/eslint-config-atlauncher/rules/plugin-react.js

@@ -25,6 +25,6 @@
     "eslint": "^3.19.0"
   },
   "engines": {
-    "node": ">= 4"
+    "node": ">= 6"
   }
 }

packages/eslint-plugin-atlauncher/package.json

@@ -1,4 +1,5 @@
-import React, { PropTypes } from 'react';
+import React from 'react';
+import PropTypes from 'proptypes';
 
 import './Button.less';