-
Notifications
You must be signed in to change notification settings - Fork 7
Update dependencies (#227) #228
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
Conversation
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.
Just two tiny nits and it's ✅ .
package.json
Outdated
"eslint-loader": "^4.0.2", | ||
"eslint-plugin-import": "^2.22.1", | ||
"eslint-plugin-jsx-a11y": "^6.4.1", | ||
"eslint-plugin-markdown": "^2.0.0-rc.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.
How about bumping this to RC2?
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.
RC2 is currently installed due to ^
. Can update it.
package.json
Outdated
"nodemon": "^2.0.6", | ||
"postcss": "^8.2.1", | ||
"postcss-loader": "^4.1.0", | ||
"markdownlint-cli2": "^0.0.13", |
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.
Let's be careful with 0.x versions, I'd leave out the ^
.
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.
Yep, I overlooked this. Thx
dc9e924
to
f1fb32d
Compare
@adamkudrna For info, there are few incompatibilities that do not allow us to update everything to the latest versions.
React 17 cannot be updated due to missing Rect 17 adapter for enzyme (will be released soon), Webpack cannot be updated because Gatsby do not suport Webpack 5. Sass loader and postcss loader are broken, I was not able to fix it.
@babel/plugin-transform-spread
must have been added to solve Gatsby issues, we need to remove it later when it will be resolved.