fix linting issues (#184)

.eslintignore

@@ -0,0 +1,3 @@
+/apps/**
+/libraries/icons/**
+node_modules

.eslintrc.yaml

@@ -15,6 +15,9 @@ rules:
   no-unused-vars:
     - error
     - varsIgnorePattern: ^React$
+  # Named exports are better
+  import/no-default-export:
+    - error
   # Not everything needs a default export, for example utils files
   import/prefer-default-export:
     - off

apps/figma-broker/index.js

@@ -88,6 +88,7 @@ async function createTokens(ctx) {
     export const tokens = {
       ${tokens.map((token) => token.name).join(',\n')}
     }
+
     `
     writeFile(
       PATHS.BASE_TOKENS,

libraries/core-react/package.json

@@ -14,7 +14,6 @@
     "url": "https://github.com/equinor/design-system"
   },
   "scripts": {
-    "lint": "eslint ./src/**/*.{jsx,js}",
     "build": "rollup -c",
     "dev": "rollup -c -w",
     "test": "jest",
@@ -30,18 +29,18 @@
   "devDependencies": {
     "@babel/cli": "^7.7.5",
     "@babel/core": "^7.7.5",
-    "@babel/preset-env": "^7.7.5",
+    "@babel/preset-env": "^7.7.6",
     "@babel/preset-react": "^7.7.4",
     "@testing-library/jest-dom": "^4.2.4",
-    "@testing-library/react": "^9.3.2",
+    "@testing-library/react": "^9.3.3",
     "babel-plugin-styled-components": "^1.10.6",
     "jest": "^24.9.0",
     "jest-styled-components": "^6.3.4",
     "prettier": "^1.19.1",
     "prop-types": "^15.7.2",
     "react": "^16.12.0",
     "react-dom": "^16.12.0",
-    "rollup": "^1.27.8",
+    "rollup": "^1.27.11",
     "rollup-plugin-babel": "^4.3.3",
     "rollup-plugin-commonjs": "^9.3.4",
     "rollup-plugin-json": "^4.0.0",

libraries/core-react/pnpmfile.js

@@ -1,17 +1,22 @@
-module.exports = {
-  hooks: {
-    readPackage,
-  },
-}
-
 function readPackage(pkg, context) {
+  const additionalDependencies = {}
+
   if (pkg.name === 'enzyme-adapter-react-16') {
-    pkg.dependencies = {
-      ...pkg.dependencies,
-      has: '^1.0.3',
-    }
+    additionalDependencies.has = '^1.0.3'
     context.log('Added missing dependencies for enzyme-adapter-react-16')
   }
 
-  return pkg
+  return {
+    ...pkg,
+    dependencies: {
+      ...pkg.dependencies,
+      ...additionalDependencies,
+    },
+  }
+}
+
+module.exports = {
+  hooks: {
+    readPackage,
+  },
 }

libraries/core-react/rollup.config.js

@@ -1,3 +1,4 @@
+/* eslint-disable import/no-default-export */
 import resolve from 'rollup-plugin-node-resolve'
 import commonjs from 'rollup-plugin-commonjs'
 import json from 'rollup-plugin-json'

libraries/core-react/rtl.setup.js

@@ -1 +1,2 @@
+/* eslint-disable import/no-extraneous-dependencies */
 import '@testing-library/jest-dom/extend-expect'

libraries/core-react/src/Button/Button.jsx

@@ -5,7 +5,7 @@ import primaryButtonTokens from '@equinor/eds-tokens/components/button/buttons-p
 import secondaryButtonTokens from '@equinor/eds-tokens/components/button/buttons-secondary.json'
 import dangerButtonTokens from '@equinor/eds-tokens/components/button/buttons-danger.json'
 import disabledButtonTokens from '@equinor/eds-tokens/components/button/buttons-disabled.json'
-import { typographyTemplate } from './../_common/templates'
+import { typographyTemplate } from '../_common/templates'
 
 const colors = {
   primary: primaryButtonTokens,
@@ -96,7 +96,7 @@ const ButtonBase = styled.button.attrs(() => ({
 // - Use ThemeProvider
 // - Set attr on button
 
-const Button = ({
+export const Button = ({
   variant,
   children,
   disabled,
@@ -146,5 +146,3 @@ Button.defaultProps = {
 }
 
 Button.displayName = 'eds-button'
-
-export default Button

libraries/core-react/src/Button/index.js

@@ -1 +1 @@
-export { default } from './Button'
+export { Button } from './Button'