Feat: Drop Node 4 support (styleguidist#839)

BREAKING CHANGE:

Node 6 is the lowest supported version.

Author: kohgpat

.travis.yml

@@ -1,12 +1,8 @@
 language: node_js
 node_js:
-  - 4
   - 6
   - 8
   - 9
-matrix:
-  allow_failures:
-    - node_js: 4
 cache:
   directories:
     - node_modules
@@ -18,15 +14,11 @@ script:
   - npm run lint
   # Run tests and Danger.js
   - |
-    if [[ "$TRAVIS_NODE_VERSION" == "4" ]]; then
-      NODE_ENV=test node --harmony-proxies $(npm bin)/jest --runInBand
+    if [[ "$TRAVIS_NODE_VERSION" == "9" ]]; then
+      npx danger ci
+      npm run test:coverage -- --runInBand
     else
-      if [[ "$TRAVIS_NODE_VERSION" == "9" ]]; then
-        npx danger ci
-        npm run test:coverage -- --runInBand
-      else
-        npm run test:jest -- --runInBand
-      fi
+      npm run test:jest -- --runInBand
     fi
   # Compile code
   - npm run compile
@@ -35,7 +27,7 @@ script:
   - npm run build:customised
   - npm run build:sections
   - |
-    if [[ "$TRAVIS_NODE_VERSION" == "8" ]]; then
+    if [[ "$TRAVIS_NODE_VERSION" == "9" ]]; then
       # Check that examples really works: no JS errors on load
       npm run test:browser:pre
       npm run test:browser

bin/.eslintrc

@@ -4,7 +4,6 @@
 		"sourceType": "script"
 	},
 	"rules": {
-		"strict": [2, "global"],
 		"es5/no-default-parameters": 2,
 		"es5/no-destructuring": 2,
 		"es5/no-spread": 2

bin/__tests__/styleguidist.spec.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const vm = require('vm');
 const path = require('path');

bin/styleguidist.js

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-'use strict';
 /* eslint-disable no-console */
 
 const minimist = require('minimist');

loaders/.eslintrc

@@ -5,7 +5,6 @@
 	},
 	"rules": {
 		"no-invalid-this": 0,
-		"strict": [2, "global"],
 		"es5/no-default-parameters": 2,
 		"es5/no-destructuring": 2,
 		"es5/no-spread": 2

loaders/css-loader.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /*
 We’re not using Webpack loader resolver because its behavior can be changed by user‘s Webpack config (Create React App
 does that for example). This way we can bypass Webpack resolver and just use Node resolver. Should be used like this:

loaders/examples-loader.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 const filter = require('lodash/filter');
 const map = require('lodash/map');

loaders/props-loader.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 const isArray = require('lodash/isArray');
 const reactDocs = require('react-docgen');

loaders/style-loader.js

@@ -4,6 +4,4 @@ does that for example). This way we can bypass Webpack resolver and just use Nod
 require('!!../../../loaders/style-loader!../../../loaders/css-loader!codemirror/lib/codemirror.css');
  */
 
-'use strict';
-
 module.exports = require('style-loader');

loaders/styleguide-loader.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const pick = require('lodash/pick');
 const commonDir = require('common-dir');
 const generate = require('escodegen').generate;

loaders/utils/chunkify.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const remark = require('remark');
 const visit = require('unist-util-visit');
 const highlightCode = require('./highlightCode');

loaders/utils/client/evalInContext.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // WARNING: This function’s source is returned by a loader without transpilation.
 // Do not use any unsupported by IE11+ features.
 

loaders/utils/client/requireInRuntime.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // WARNING: This function’s source is returned by a loader without transpilation.
 // Do not use any unsupported by IE11+ features.
 

loaders/utils/expandDefaultComponent.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const COMPONENT_PLACEHOLDER = '__COMPONENT__';
 const COMPONENT_PLACEHOLDER_REGEXP = new RegExp(COMPONENT_PLACEHOLDER, 'g');
 

loaders/utils/filterComponentsWithExample.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Filter out components without an example file.
  *

loaders/utils/getAllContentPages.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Get all section content pages.
  *

loaders/utils/getComponentFiles.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 const glob = require('glob');
 const isFunction = require('lodash/isFunction');

loaders/utils/getComponentFilesFromSections.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const getComponentFiles = require('./getComponentFiles');
 
 /**

loaders/utils/getComponentPatternsFromSections.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Return all glob patterns from all sections.
  *

loaders/utils/getComponents.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const processComponent = require('./processComponent');
 
 /**

loaders/utils/getExamples.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 const requireIt = require('./requireIt');

loaders/utils/getNameFromFilePath.js

@@ -1,4 +1,3 @@
-'use strict';
 const path = require('path');
 const _ = require('lodash');
 

loaders/utils/getProps.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 const fs = require('fs');
 const reactDocs = require('react-docgen');

loaders/utils/getRequires.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // Need to supply the regex test as a string for reuse in unit tests
 // Currently, trying to change flags throws a TypeError
 // Slated for change in ES6, but not possible now:

loaders/utils/getSections.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // This two functions should be in the same file because of cyclic imports
 
 const fs = require('fs');

loaders/utils/highlightCode.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const hljs = require('highlight.js');
 
 /**

loaders/utils/highlightCodeInMarkdown.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const remark = require('remark');
 const visit = require('unist-util-visit');
 const highlightCode = require('./highlightCode');

loaders/utils/parseExample.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const consts = require('../../scripts/consts');
 const lowercaseKeys = require('lowercase-keys');
 

loaders/utils/processComponent.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 const getNameFromFilePath = require('./getNameFromFilePath');

loaders/utils/removeDoclets.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // Doclet regexp from react-docgen
 const DOCLET_REGEXP = /^@(\w+)(?:$|\s((?:[^](?!^@\w))*))/gim;
 

loaders/utils/requireIt.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const b = require('ast-types').builders;
 
 /**

loaders/utils/slugger.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const GithubSlugger = require('github-slugger');
 
 // Export the singleton instance of GithubSlugger

loaders/utils/sortProps.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const sortBy = require('lodash/sortBy');
 
 /**

package.json

@@ -26,7 +26,7 @@
     "scripts"
   ],
   "engines": {
-    "node": ">=4",
+    "node": ">=6",
     "npm": ">=3"
   },
   "dependencies": {

scripts/.eslintrc

@@ -4,7 +4,6 @@
 		"sourceType": "script"
 	},
 	"rules": {
-		"strict": [2, "global"],
 		"es5/no-default-parameters": 2,
 		"es5/no-destructuring": 2,
 		"es5/no-spread": 2

scripts/__mocks__/build.js

@@ -1,5 +1,3 @@
-'use strict';
-
 module.exports = function server(config, callback) {
 	callback(null, { stats: true });
 	return {};

scripts/__mocks__/server.js

@@ -1,5 +1,3 @@
-'use strict';
-
 module.exports = function server(config, callback) {
 	callback(null);
 	return {};

scripts/build.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const webpack = require('webpack');
 const makeWebpackConfig = require('./make-webpack-config');
 

scripts/config.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 const findup = require('findup');

scripts/consts.js

@@ -1,5 +1,3 @@
-'use strict';
-
 module.exports = {
 	HOMEPAGE: 'https://react-styleguidist.js.org/',
 	BUGS: 'https://github.com/styleguidist/react-styleguidist/issues',

scripts/create-server.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const webpack = require('webpack');
 const WebpackDevServer = require('webpack-dev-server');
 const merge = require('webpack-merge');

scripts/index.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // Make sure user has webpack installed
 require('./utils/ensureWebpack');
 

scripts/logger.js

@@ -1,4 +1,3 @@
-'use strict';
 /* eslint-disable no-console */
 
 const _ = require('lodash/fp');

scripts/make-webpack-config.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 const webpack = require('webpack');
 const UglifyJSPlugin = require('uglifyjs-webpack-plugin');

scripts/schemas/config.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // If you want to access any of these options in React, don’t forget to update CLIENT_CONFIG_OPTIONS array
 // in loaders/styleguide-loader.js
 

scripts/server.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const createServer = require('./create-server');
 
 module.exports = function server(config, callback) {

scripts/utils/StyleguidistOptionsPlugin.js

@@ -1,5 +1,3 @@
-'use strict';
-
 // Webpack plugin that makes Styleguidist config available for Styleguidist webpack loaders.
 // It will be available as `this._styleguidist`.
 //

scripts/utils/ensureWebpack.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Check webpack availability and version at run time instead of using peerDependencies to allow
  * usage of build tools that contains webpack as their own dependency, like Create React App.

scripts/utils/error.js

@@ -1,5 +1,3 @@
-'use strict';
-
 class StyleguidistError extends Error {
 	constructor(message, extra) {
 		super(message);

scripts/utils/findFileCaseInsensitive.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 const memoize = require('lodash/memoize');

scripts/utils/findUserWebpackConfig.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 

scripts/utils/getUserPackageJson.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const path = require('path');
 
 /**

scripts/utils/getWebpackVersion.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const semverUtils = require('semver-utils');
 
 /**

scripts/utils/mergeWebpackConfig.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const isFunction = require('lodash/isFunction');
 const omit = require('lodash/omit');
 const mergeBase = require('webpack-merge');

scripts/utils/sanitizeConfig.js

@@ -1,5 +1,3 @@
-'use strict';
-
 const fs = require('fs');
 const path = require('path');
 const isDirectory = require('is-directory');