Skip to content

Commit

Permalink
lint: Upgrade ESLint to ^6.8.0.
Browse files Browse the repository at this point in the history
We're not quite ready for ESLint 7 because several of our plugins
don't support it yet. Fair enough; 6.8.0 is the latest on ESLint 6.

Apart from that, I was aware of some specific minimum versions of
things we'll need, so I just made those tweaks on that basis:

- `prettier-eslint` at 10.1.1 is the minimum that I'm confident has
  full ESLint 6 support; see my comment there at
  prettier/prettier-eslint#338 (comment).

- `eslint-config-airbnb` at 18.1.0, I'd discovered earlier, has
  6.8.0 as a peer dep. I checked to see if 18.1.0 is the latest
  version, and it is.

Otherwise, I just followed the trail of peer dependency warnings
until I could run `rm -rf node_modules && yarn` without any
warnings. It's likely that this would have given the same outcome
for `prettier-eslint` and `eslint-config-airbnb`, but it's not what
I did for those.

Fixes: zulip#4120
  • Loading branch information
chrisbobbe committed Jul 13, 2020
1 parent d6b0a5a commit 89d34f0
Show file tree
Hide file tree
Showing 2 changed files with 554 additions and 164 deletions.
11 changes: 6 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -98,15 +98,16 @@
"babel-eslint": "^10.0.3",
"babel-plugin-transform-remove-console": "^6.9.4",
"deep-freeze": "^0.0.1",
"eslint": "^5.16.0",
"eslint-config-airbnb": "^17.1.1",
"eslint": "^6.8.0",
"eslint-config-airbnb": "^18.1.0",
"eslint-config-prettier": "^6.0.0",
"eslint-plugin-flowtype": "^3.9.0",
"eslint-plugin-import": "^2.18.1",
"eslint-plugin-import": "^2.20.1",
"eslint-plugin-jest": "^22.11.1",
"eslint-plugin-jsx-a11y": "^6.2.3",
"eslint-plugin-prettier": "^3.1.0",
"eslint-plugin-react": "^7.14.2",
"eslint-plugin-react": "^7.19.0",
"eslint-plugin-react-hooks": "2.5.0",
"flow-bin": "^0.105.0",
"flow-coverage-report": "^0.6.0",
"flow-typed": "^2.4.0",
Expand All @@ -119,7 +120,7 @@
"lolex": "^5.1.1",
"metro-react-native-babel-preset": "^0.56.0",
"prettier": "^1.18.2",
"prettier-eslint": "^9.0.0",
"prettier-eslint": "^10.1.1",
"prettier-eslint-cli": "^5.0.0",
"prop-types": "^15.7.2",
"react-native-cli": "^2.0.1",
Expand Down
Loading

0 comments on commit 89d34f0

Please sign in to comment.