Commit fc32623f authored by Dan Abramov's avatar Dan Abramov
Browse files

Replace eslint-plugin-flow-vars with eslint-plugin-flowtype

This is the one people are converging on, and flow-vars may get deprecated.
parent 9dab8715
No related merge requests found
Showing with 7 additions and 6 deletions
+7 -6
...@@ -22,7 +22,7 @@ module.exports = { ...@@ -22,7 +22,7 @@ module.exports = {
parser: 'babel-eslint', parser: 'babel-eslint',
// import plugin is termporarily disabled, scroll below to see why // import plugin is termporarily disabled, scroll below to see why
plugins: [/*'import', */'flow-vars', 'jsx-a11y', 'react'], plugins: [/*'import', */'flowtype', 'jsx-a11y', 'react'],
env: { env: {
browser: true, browser: true,
...@@ -197,8 +197,9 @@ module.exports = { ...@@ -197,8 +197,9 @@ module.exports = {
'jsx-a11y/img-redundant-alt': 'warn', 'jsx-a11y/img-redundant-alt': 'warn',
'jsx-a11y/no-access-key': 'warn', 'jsx-a11y/no-access-key': 'warn',
// https://github.com/zertosh/eslint-plugin-flow-vars // https://github.com/gajus/eslint-plugin-flowtype
'flow-vars/define-flow-type': 'warn', 'flowtype/define-flow-type': 'warn',
'flow-vars/use-flow-type': 'warn' 'flowtype/require-valid-file-annotation': 'warn',
'flowtype/use-flow-type': 'warn'
} }
}; };
...@@ -257,7 +257,7 @@ If you don’t need ESLint integration with your editor, you can safely delete t ...@@ -257,7 +257,7 @@ If you don’t need ESLint integration with your editor, you can safely delete t
Finally, you will need to install some packages *globally*: Finally, you will need to install some packages *globally*:
```sh ```sh
npm install -g eslint babel-eslint eslint-plugin-react eslint-plugin-import eslint-plugin-jsx-a11y eslint-plugin-flow-vars npm install -g eslint babel-eslint eslint-plugin-react eslint-plugin-import eslint-plugin-jsx-a11y eslint-plugin-flowtype
``` ```
We recognize that this is suboptimal, but it is currently required due to the way we hide the ESLint dependency. The ESLint team is already [working on a solution to this](https://github.com/eslint/eslint/issues/3458) so this may become unnecessary in a couple of months. We recognize that this is suboptimal, but it is currently required due to the way we hide the ESLint dependency. The ESLint team is already [working on a solution to this](https://github.com/eslint/eslint/issues/3458) so this may become unnecessary in a couple of months.
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment