-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(FEC-10290): upgrade NPM packages #109
Conversation
@@ -13,7 +13,9 @@ | |||
"should": true, | |||
"sinon": true, | |||
"__VERSION__": true, | |||
"__NAME__": true | |||
"__NAME__": true, | |||
"process": true, |
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.
what is that added for?
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.
karma.config
and webpack.config
was removed from .eslintignore
(eslint should lint those files too...) and they are using Node globals that you need to define to eslint so it won't detect them as errors.
…dash into upgrade-npm-packages
Description of the Changes
✅ eslint upgraded to 7.x.x
✅ babel upgraded to @babel (7.x.x)
✅ webpack upgraded to 4.x.x
CheckLists