Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update & Optimize CI configs #473

Merged
merged 1 commit into from
Jul 31, 2017
Merged

Update & Optimize CI configs #473

merged 1 commit into from
Jul 31, 2017

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Jul 31, 2017

  • Test on Node 8
  • Only run unit tests on Appveyor
  • Optimize TravisCI builds to use the minimum number of containers while keeping the performance reasonable

@RyanZim
Update & Optimize CI configs
5435252 
- Test on Node 8
- Only run unit tests on Appveyor
- Optimize TravisCI builds to use the minimum number of containers
  while keeping the performance reasonable
@RyanZim RyanZim requested review from jprichardson and manidlou July 31, 2017 19:23
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.364% when pulling 5435252 on ci into 9b5718c on master.

@RyanZim
Copy link
Collaborator Author

RyanZim commented Jul 31, 2017

Travis takes somewhere between 1-1.5 min. to run; AppVeyor still takes ~3 min. due to no concurrent builds.

@jprichardson jprichardson merged commit ab2fae9 into master Jul 31, 2017
@RyanZim RyanZim deleted the ci branch July 31, 2017 20:07
@renovate renovate bot mentioned this pull request Feb 14, 2018
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jprichardson jprichardson Awaiting requested review from jprichardson

@manidlou manidlou Awaiting requested review from manidlou

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

Successfully merging this pull request may close these issues.
3 participants
@RyanZim @coveralls @jprichardson