fix(eslint): add config and docs to lint test, fix errors

config/jest.config.js

@@ -1,10 +1,10 @@
-const path = require('path');
+const path = require('path')
 
 module.exports = {
   rootDir: path.resolve('src'),
   moduleNameMapper: {
-    "\\.modules\.scss$": "identity-obj-proxy",
-    "^.+\\.(css|scss)$": "<rootDir>/__mocks__/styleMock.js"
+    '\\.modules.scss$': 'identity-obj-proxy',
+    '^.+\\.(css|scss)$': '<rootDir>/__mocks__/styleMock.js'
   },
   setupTestFrameworkScriptFile: path.resolve('node_modules/jest-enzyme/lib/index.js')
-};
+}

config/rollup.config.js

@@ -1,21 +1,23 @@
-import path from 'path';
+import path from 'path'
 
-import resolve from 'rollup-plugin-node-resolve';
-import commonjs from 'rollup-plugin-commonjs';
+import resolve from 'rollup-plugin-node-resolve'
+import commonjs from 'rollup-plugin-commonjs'
 
-import babel from 'rollup-plugin-babel';
+import babel from 'rollup-plugin-babel'
 
-import postcss from 'rollup-plugin-postcss';
-import postcssModules from 'postcss-modules';
-import sass from 'node-sass';
-import autoprefixer from 'autoprefixer';
+import postcss from 'rollup-plugin-postcss'
+import postcssModules from 'postcss-modules'
+import sass from 'node-sass'
+import autoprefixer from 'autoprefixer' // eslint-disable-line import/no-extraneous-dependencies
 
-const cssExportMap = {};
+const cssExportMap = {}
 
+/* eslint-disable no-shadow */
 const sassPreprocessor = (content, id) => new Promise((resolve) => {
-  const result = sass.renderSync({ file: id });
-  resolve({ code: result.css.toString() });
-});
+  const result = sass.renderSync({ file: id })
+  resolve({ code: result.css.toString() })
+})
+/* eslint-enable */
 
 export default {
   entry: path.resolve('./src/index.js'),
@@ -44,19 +46,19 @@ export default {
         postcssModules({
           globalModulePaths: [/src\/scss/, /src\/components\/(?!Button)/],
           generateScopedName: 'TDS_[name]__[local]___[hash:base64:5]',
-          getJSON (id, exportTokens) {
-            cssExportMap[id] = exportTokens;
+          getJSON(id, exportTokens) {
+            cssExportMap[id] = exportTokens
           }
         })
       ],
       getExportNamed: false,
-      getExport (id) {
-        return cssExportMap[id];
+      getExport(id) {
+        return cssExportMap[id]
       }
     }),
     babel({
       plugins: ['external-helpers'],
       exclude: 'node_modules/**'
     })
   ]
-};
+}

config/styleguide.config.js

@@ -1,42 +1,42 @@
-const path = require('path');
+const path = require('path')
 
-const version = require('../package.json').version;
+const version = require('../package.json').version
 
-const toggle = (path) => (
+const toggle = path => ( // eslint-disable-line no-shadow
   process.env.NODE_ENV === 'production' ? undefined : path
-);
+)
 
-const compact = (array) => array.filter((element) => element !== undefined);
+const compact = array => array.filter(element => element !== undefined)
 
 
 module.exports = {
   title: `TDS v${version}`,
 
   skipComponentsWithoutExample: true,
   getComponentPathLine(componentPath) {
-    let name = path.basename(componentPath, '.jsx');
+    let name = path.basename(componentPath, '.jsx')
 
     // Grid has name-spaced sub-components
     if (path.dirname(componentPath).includes('Grid')) {
-      name = 'Grid';
+      name = 'Grid'
     }
 
     // Steps has name-spaced sub-components
     if (path.dirname(componentPath).includes('StepTracker')) {
-      name = 'Steps';
+      name = 'Steps'
     }
 
     // ExpandCollapse has name-spaced sub-components
     if (path.dirname(componentPath).includes('ExpandCollapse')) {
-      name = 'ExpandCollapse';
+      name = 'ExpandCollapse'
     }
 
     // Link component has name-space sub-components
     if (path.dirname(componentPath).includes('Link')) {
-      name = 'Link';
+      name = 'Link'
     }
 
-    return `import { ${name} } from '@telusdigital/tds';`;
+    return `import { ${name} } from '@telusdigital/tds';`
   },
 
   showUsage: true,
@@ -147,7 +147,7 @@ module.exports = {
           ]
         },
         {
-          name: "Icons",
+          name: 'Icons',
           components: path.resolve('src/components/Icon/Icon.jsx')
         },
         {
@@ -228,4 +228,4 @@ module.exports = {
       linkHover: '#54595F'
     }
   }
-};
+}

docs/components/Logo.jsx

@@ -1,7 +1,7 @@
-import React from 'react';
+import React from 'react'
 
-import {version} from '../../package.json';
-import logo from './Logo.svg';
+import { version } from '../../package.json'
+import logo from './Logo.svg'
 
 const Logo = () => (
   <div>
@@ -11,6 +11,6 @@ const Logo = () => (
 
     TDS v{version}
   </div>
-);
+)
 
-export default Logo;
+export default Logo

package.json

@@ -20,7 +20,7 @@
     "deploy:docs": "node ./scripts/deploy-docs.js",
     "deploy:package": "scripts/publish.sh",
     "dev": "styleguidist server --config config/styleguide.config.js",
-    "lint:js": "eslint src --ext '.js,.jsx' --config config/.eslintrc.json --fix",
+    "lint:js": "eslint src config docs --ext '.js,.jsx' --config config/.eslintrc.json --fix",
     "lint:scss": "stylelint 'src/scss/**/*.scss' --config config/.stylelintrc.json",
     "lint:ec": "echint",
     "lint": "yarn lint:js && yarn lint:scss && yarn lint:ec",