-
-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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(core): update css-loader to latest #4619
Conversation
Get latest from storybook
merge 4.0.0-rc.1 into local
Update from latest master
…ve lodash dependencies
Codecov Report
@@ Coverage Diff @@
## master #4619 +/- ##
=======================================
Coverage 35.59% 35.59%
=======================================
Files 557 557
Lines 6732 6732
Branches 884 884
=======================================
Hits 2396 2396
Misses 3876 3876
Partials 460 460 Continue to review full report at Codecov.
|
lib/core/package.json
Outdated
@@ -47,7 +47,7 @@ | |||
"commander": "^2.19.0", | |||
"common-tags": "^1.8.0", | |||
"core-js": "^2.5.7", | |||
"css-loader": "^1.0.0", | |||
"css-loader": "^1.0.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.
Looks like css-loader
is used in some other packages too. Can you update it there as well?
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.
Sure thing, happy to!
…dency CVE on lodash modularized packages
update from master
update from master
Issue:
css-loader 1.0.0 depends on vulnerable versions of lodash packages
What I did
updated to latest css-loader that addresses these transitive dependency issues
How to test
yarn test