-
Notifications
You must be signed in to change notification settings - Fork 687
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
Remove lodash from create-pwa #3003
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.
f007005
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.
this should work - now we just need to test it.
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.
Working when used against node packages/create-pwa/bin/create-pwa
. Need to publish check against yarn create @magento/pwa
. Thanks for working on this @jimbo.
Ignore the coveralls build, it triggered because of push event. @jimbo Please check on test failure though. |
Description
Remove
lodash
from@magento/create-pwa
.Related Issue
PWA-1452
Acceptance
Verification Stakeholders
Specification
Verification Steps
yarn create
should workScreenshots / Screen Captures (if appropriate)
Checklist