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

Enable Yarn check files #3769

Merged
merged 2 commits into from
Jan 13, 2018
Merged

Enable Yarn check files #3769

merged 2 commits into from
Jan 13, 2018

Conversation

Timer
Copy link
Contributor

@Timer Timer commented Jan 13, 2018

Adds ~2-3 seconds to install time, but can save some headaches.
Pretty important when multiple users start fiddling with package.json files and we start using more advanced package manager settings.

Also will help recover your tree after you use npm by accident without wiping out all node_modules.

Adds approx 4 seconds to install time, but can save some headaches.
@Timer Timer changed the base branch from next to master January 13, 2018 01:09
@Timer Timer merged commit 238af4b into facebook:master Jan 13, 2018
@Timer Timer deleted the enable-check-files branch January 13, 2018 03:14
@gaearon gaearon mentioned this pull request Jan 15, 2018
Pavek pushed a commit to Pavek/create-react-app that referenced this pull request Jul 10, 2018
* This is a good default.
Adds approx 4 seconds to install time, but can save some headaches.

* Add no lockfile for add, too
@lock lock bot locked and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants