Skip to content
This repository has been archived by the owner on Jan 20, 2020. It is now read-only.

chore(npm): generate package lock #57

Closed
wants to merge 2 commits into from
Closed

chore(npm): generate package lock #57

wants to merge 2 commits into from

Conversation

awitherow
Copy link
Contributor

To help better with dependency management.

@awitherow awitherow mentioned this pull request Jun 6, 2017
@fb55
Copy link
Contributor

fb55 commented Jun 6, 2017

I'm not a fan of having dependency lock files on modules. While they make a lot of sense for products having them for modules will prevent the package manager from de-duplicating dependencies. Also, if we support the npm format we should also support the yarn format, which will become unmanageable at some point.

@fb55 fb55 closed this Jun 6, 2017
@awitherow
Copy link
Contributor Author

Good to know!

@awitherow awitherow deleted the npm-shrinkwrap branch June 7, 2017 05:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants