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.

Sign up for GitHub

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

Jump to bottom

Travis: Use Yarn #355

Merged
merged 2 commits into from
Sep 4, 2017
Merged

Travis: Use Yarn #355

merged 2 commits into from
Sep 4, 2017

Conversation

lukeed
Copy link
Member

@lukeed lukeed commented Sep 4, 2017 •
edited
Loading

Node 6 Node 8 Node 8 (serve) Total
Current 17m 45s 11m 27s 178.8s 29m 12s
With Yarn 17m 44s 8m 10s 126.6s 25m 54s

Not sure why Node 6 had no change 🤔 Will re-run to see if makes a difference on 2nd run

Update: On 2nd run, Node6 completed in 14m 1s, reducing total time to 22m 11s!

Closes #350

@reznord reznord merged commit 0be97e8 into master Sep 4, 2017
@lukeed lukeed deleted the use-yarn branch September 4, 2017 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reznord reznord reznord approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lukeed @reznord