@sb1/eslint-config-ffe 8.0.3 → 8.1.3

package/index.js

@@ -1,17 +1,14 @@
 module.exports = {
-    'extends': [
+    extends: [
         '@sb1/eslint-config-ffe-base',
         './rules/jsx',
-        './rules/jsx-a11y'
+        './rules/jsx-a11y',
     ].map(require.resolve),
-    'plugins': [
-        'react',
-        'jsx-a11y'
-    ],
-    'parserOptions': {
-        'ecmaFeatures': {
-            'jsx': true
+    plugins: ['react', 'jsx-a11y', 'react-hooks'],
+    parserOptions: {
+        ecmaFeatures: {
+            jsx: true,
         },
     },
-    'rules': {}
+    rules: {},
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sb1/eslint-config-ffe",
-  "version": "8.0.3",
+  "version": "8.1.3",
   "description": "",
   "license": "MIT",
   "author": "SpareBank 1",
@@ -17,21 +17,23 @@
     "lint": "eslint ."
   },
   "dependencies": {
-    "@sb1/eslint-config-ffe-base": "^3.0.3"
+    "@sb1/eslint-config-ffe-base": "^3.0.4"
   },
   "devDependencies": {
     "eslint": "^5.9.0",
-    "eslint-plugin-import": "^2.0.1",
+    "eslint-plugin-import": "^2.20.1",
     "eslint-plugin-jsx-a11y": "^6.0.3",
-    "eslint-plugin-react": "^7.0.0"
+    "eslint-plugin-react": "^7.0.0",
+    "eslint-plugin-react-hooks": "^2.0.1"
   },
   "peerDependencies": {
-    "eslint": "^5.9.0",
-    "eslint-plugin-jsx-a11y": "^6.0.3",
-    "eslint-plugin-react": ">=6"
+    "eslint": "*",
+    "eslint-plugin-jsx-a11y": "*",
+    "eslint-plugin-react": "*",
+    "eslint-plugin-react-hooks": "*"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "cfa663a17d95dd166654579832ebdf3deaad7400"
+  "gitHead": "2bbaa979bf25dc291f521354217b28b17c04a270"
 }

package/rules/jsx-a11y.js

@@ -67,7 +67,7 @@ module.exports = {
 
         // Enforce that elements with explicit or implicit roles defined contain only aria-* properties supported by that role.
         // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/role-supports-aria-props.md
-        'jsx-a11y/role-supports-aria-props': 2,
+        'jsx-a11y/role-supports-aria-props': 1,
 
         // Enforce tabIndex value is not greater than zero.
         // https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/tabindex-no-positive.md

package/rules/jsx.js

@@ -1,8 +1,8 @@
 module.exports = {
-    'rules': {
+    rules: {
         // Enforce boolean attributes notation in JSX
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-boolean-value.md
-        'react/jsx-boolean-value': [2, 'always' ],
+        'react/jsx-boolean-value': [2, 'always'],
 
         // Validate JSX has key prop when in array or iterator
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-key.md
@@ -30,11 +30,11 @@ module.exports = {
 
         // Ensure correct position of the first property.
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-first-prop-new-line.md
-        'react/jsx-first-prop-new-line' : [2, "multiline"],
+        'react/jsx-first-prop-new-line': [2, 'multiline'],
 
         // Enforce the closing bracket location for JSX multiline elements.
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-closing-bracket-location.md
-        'react/jsx-closing-bracket-location' : [2],
+        'react/jsx-closing-bracket-location': [2],
 
         // Prevent usage of dangerous JSX properties
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-danger.md
@@ -58,7 +58,7 @@ module.exports = {
 
         // Prevent multiple component definition per file
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-multi-comp.md
-        'react/no-multi-comp': [2, {'ignoreStateless': true}],
+        'react/no-multi-comp': [2, { ignoreStateless: true }],
 
         // Prevent usage of unknown DOM property
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-unknown-property.md
@@ -86,6 +86,14 @@ module.exports = {
 
         // Prevent missing props validation in a React component definition
         // https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/prop-types.md
-        'react/prop-types': [2]
-    }
+        'react/prop-types': [2],
+
+        // Prevent breaking rules of hooks
+        // https://reactjs.org/docs/hooks-rules.html
+        'react-hooks/rules-of-hooks': 'error',
+
+        // Warn about hook usage that is probably wrong
+        // https://reactjs.org/docs/hooks-rules.html
+        'react-hooks/exhaustive-deps': 'warn',
+    },
 };

package/CHANGELOG.md

@@ -1,193 +0,0 @@
-# Change Log
-
-All notable changes to this project will be documented in this file.
-See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
-
-## [8.0.3](https://github.com/SpareBank1/designsystem/compare/@sb1/eslint-config-ffe@8.0.2...@sb1/eslint-config-ffe@8.0.3) (2018-12-04)
-
-**Note:** Version bump only for package @sb1/eslint-config-ffe
-
-<a name="8.0.2"></a>
-
-## [8.0.2](https://github.com/SpareBank1/designsystem/compare/@sb1/eslint-config-ffe@8.0.1...@sb1/eslint-config-ffe@8.0.2) (2018-08-15)
-
-**Note:** Version bump only for package @sb1/eslint-config-ffe
-
-<a name="8.0.1"></a>
-
-## [8.0.1](https://github.com/SpareBank1/designsystem/compare/@sb1/eslint-config-ffe@8.0.0...@sb1/eslint-config-ffe@8.0.1) (2018-08-09)
-
-**Note:** Version bump only for package @sb1/eslint-config-ffe
-
-<a name="8.0.0"></a>
-
-# 8.0.0 (2018-02-25)
-
-### Bug Fixes
-
--   Replace eslint ignore with customized lint rule. ([d98ed78](https://github.com/SpareBank1/designsystem/commit/d98ed78))
-
-### Chores
-
--   add [@sb1](https://github.com/sb1)/ scope to all packages ([37efbb4](https://github.com/SpareBank1/designsystem/commit/37efbb4))
-
-### Features
-
--   **eslint-config-ffe:** Add contribute.md link to readme ([68645d6](https://github.com/SpareBank1/designsystem/commit/68645d6))
--   **eslint-config-ffe:** Upgrade to eslint 4 ([cd8b1c6](https://github.com/SpareBank1/designsystem/commit/cd8b1c6))
-
-### BREAKING CHANGES
-
--   All packages have been renamed to add the @sb1 scope.
-
-See links [1] and [2] to read more about package scopes.
-
-Unfortunately this requires you update both your dependencies in
-`package.json` and all `import` or `require` statements in your code.
-Also, you unfortunately have to upgrade all FFE packages at once due to
-dependency between packages. If you are on the latest version prior to
-this breaking change the upgrade should be a matter of updating
-`package.json` with new package names and versions, and updating your
-imports.
-
-To help find the proper package versions you can either browse to the
-packages you need or check out the list of Git tags either on
-Github [3] or by cloning the repository and running the `git tag`
-command. All package versions get a tag in the format
-_package@version_.
-
-Update your `package.json` with the latest package version and add the
-@sb1 scope to the package name:
-
-```diff
-- "ffe-core": "11.0.2",
-+ "@sb1/ffe-core": "11.0.2",
-```
-
-Scoped packages all get put in a subdirectory in `node_modules/`. In our
-case packages will be put in the `node_modules/@sb1/` directory. If your
-build depends on file paths (for instance for copying fonts) you need to
-update that path to include the scope.
-
-The directory structure also means you have to update your imports, both
-in Less and in JavaScript.
-
-Using `less-plugin-npm-import`:
-
-```diff
-- @import 'npm://ffe-core/less/ffe';
-+ @import 'npm://@sb1/ffe-core/less/ffe';
-```
-
-Using Webpack:
-
-```diff
-- @import '~ffe-core/less/ffe';
-+ @import '~@sb1/ffe-core/less/ffe';
-```
-
-Using plain old paths:
-
-```diff
-- @import '../path/to/node_modules/ffe-core/less/ffe';
-+ @import '../path/to/node_modules/@sb1/ffe-core/less/ffe';
-```
-
-JavaScript `import`:
-
-```diff
-- import { ActionButton } from 'ffe-buttons-react';
-+ import { ActionButton } from '@sb1/ffe-buttons-react';
-```
-
-JavaScript `require`:
-
-```diff
-- const { ActionButton } = require('ffe-buttons-react').default;
-+ const { ActionButton } = require('@sb1/ffe-buttons-react').default;
-```
-
-[1]: https://docs.npmjs.com/misc/scope
-[2]: https://docs.npmjs.com/getting-started/scoped-packages
-[3]: https://github.com/sparebank1/designsystem/tags
-
--   **eslint-config-ffe:** Removes deprecated ecmaFeature setting and drops
-    support for eslint 3 and older.
-
-<a name="7.0.2"></a>
-
-## [7.0.2](https://github.com/SpareBank1/designsystem/compare/eslint-config-ffe@7.0.1...eslint-config-ffe@7.0.2) (2018-02-22)
-
-**Note:** Version bump only for package eslint-config-ffe
-
-<a name="7.0.1"></a>
-
-## [7.0.1](https://github.com/SpareBank1/designsystem/compare/eslint-config-ffe@7.0.0...eslint-config-ffe@7.0.1) (2018-01-10)
-
-### Bug Fixes
-
--   Replace eslint ignore with customized lint rule.
-
-<a name="7.0.0"></a>
-
-# 7.0.0 (2017-12-22)
-
-### Features
-
--   **eslint-config-ffe:** Add contribute.md link to readme
--   **eslint-config-ffe:** Upgrade to eslint 4
-
-### BREAKING CHANGES
-
--   **eslint-config-ffe:** Removes deprecated ecmaFeature setting and drops
-    support for eslint 3 and older.
-
-## 6.0.2 (13.10.2017)
-
--   (patch) added support for `eslint-plugin-react@7`
--   (patch) added various devDependencies
-
-## 6.0.1 (08.03.2017)
-
--   (patch) at least one rule in `eslint-plugin-react` no longer work without an array prop.
-
-# 6.0.0 (29.12.2016)
-
--   (breaking) jsx-a11y rules are included by default and require eslint-plugin-jsx-a11y.
-
-# 5.1.0 (28.12.2016)
-
--   (feature) Include rules for eslint-plugin-jsx-a11y. Consumers must explicit included the rules.
-
-## 5.0.1 (18.10.2016)
-
--   (patch) Remove eslint-plugin-import from peerDependencies, we get that from eslint-config-ffe-base. To avoid having to maintain the version number in two places.
-
-# 5.0.0 (04.10.2016)
-
--   (patch) Add eslint@3.0.0 to peerDependencies
--   (patch) Update eslint-plugin-import to 1.10.1 to get updated peerDependencies
--   (breaking) Add elinst-plugin-react@6.0.0 to peerDependencies
-
-# 4.0.0 (20.09.2016)
-
--   (breaking) Enforce line brake after first prop in components if multiline props
--   (breaking) Enforce closing tag of components to be on new line if multiline props
-
-# 3.0.0 (10.08.2016)
-
--   (breaking) Add new version of eslint-config-ffe-base which removes the "no-eq-null"-rule
-
-# 2.0.0 (12.05.2016)
-
--   (breaking) Move most rules to eslint-config-ffe-base
-
-## 1.0.1 (06.05.2016)
-
--   (bugfix) Removed duplicate rule "no-duplicates"
--   (feature) Added "prop-types" rule
--   (bugfix) Added "ignoreStateless"-option to "no-multi-comp"-rule
-
-# 1.0.0 (02.05.2016)
-
--   Initial release