Skip to content
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

Library should not contains lockfile #321

Merged
merged 1 commit into from
Nov 8, 2016

Conversation

fengmk2
Copy link
Contributor

@fengmk2 fengmk2 commented Nov 7, 2016

@TooTallNate
Copy link
Contributor

This makes sense to me 👍

@thebigredgeek
Copy link
Contributor

Hmm. Yeah good points all around. Can you add to .gitignore and .npmignore please @fengmk2 ?

@fengmk2
Copy link
Contributor Author

fengmk2 commented Nov 8, 2016

@thebigredgeek Added!

@thebigredgeek
Copy link
Contributor

@fengmk2 thanks merging. As a note, if you are using Yarn, use pure mode to avoid caching in lockfiles

@thebigredgeek thebigredgeek merged commit 2f9dc36 into debug-js:master Nov 8, 2016
@fengmk2 fengmk2 deleted the remove-lockfile branch November 8, 2016 04:48
@fengmk2
Copy link
Contributor Author

fengmk2 commented Nov 8, 2016

@thebigredgeek yep, I know how yarn work.

@thebigredgeek
Copy link
Contributor

Kk. Just trying to be helpful :D. I didn't know about that option until I read up more on it. Been busy with work. Started using Yarn to simply save time during build debugging haha

@fengmk2
Copy link
Contributor Author

fengmk2 commented Nov 8, 2016

Yep, yarn install is very fast after lockfile created.
We're using npminstall to doing the same job, and it don't need lockfile but install still very fast!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants