-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: bump dev dependencies #1955
Conversation
@@ -21,5 +21,7 @@ module.exports = { | |||
plugins: [ | |||
'@babel/plugin-proposal-class-properties', | |||
'@babel/plugin-proposal-object-rest-spread', | |||
'@babel/plugin-proposal-nullish-coalescing-operator', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Otherwise jest won't be able to handle TS 3.7 nullish and optional chaining later on. (babel-preset-typescript doesnt include that)
"eslint": "4.x", | ||
"eslint-config-airbnb": "17.1.0", | ||
"eslint-config-prettier": "^2.9.0", | ||
"babel-eslint": "^10.0.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know why its been so long since last bump :o
"jest": "^24.9.0", | ||
"lerna": "^3.18.1", | ||
"lerna-changelog": "^0.8.2", | ||
"lint-staged": "^7.2.0", | ||
"picomatch": "^2.1.0", | ||
"prettier": "^1.18.2", | ||
"prettier": "^1.19.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TS 3.7 support
@@ -33,6 +33,10 @@ module.exports = { | |||
'react/button-has-type': OFF, // 1 | |||
'react/forbid-prop-types': OFF, // 1 | |||
'react/require-default-props': OFF, // 1 | |||
'jsx-a11y/control-has-associated-label': OFF, // 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of the eslint plugins are updated and stricter. So turning it off
@@ -75,5 +75,11 @@ | |||
"tiny-lr": "^1.1.1", | |||
"tree-node-cli": "^1.2.5", | |||
"truncate-html": "^1.0.3" | |||
}, | |||
"devDependencies": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of this is specific to v1 only
Deploy preview for docusaurus-preview ready! Built with commit 2e9999f |
Deploy preview for docusaurus-2 ready! Built with commit 2e9999f |
Motivation
Bump dev dependencies such as eslint, etc. Also moved some dev deps to its respective package. Root dev deps should be more for something like
lerna, @types, and eslint
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
CI should all pass