Skip to content

Speed up Travis#386

Merged
rnorth merged 2 commits intomasterfrom
speed_up_travis
Jul 1, 2017
Merged

Speed up Travis#386
rnorth merged 2 commits intomasterfrom
speed_up_travis

Conversation

@bsideup
Copy link
Copy Markdown
Member

@bsideup bsideup commented Jun 28, 2017

What I did:

  • removed custom Docker installation (with the latest Travis release they have Docker 2017.03-ce)
  • merged alpine + shade tests (-1 allocated node. Travis limits it to 5)
  • removed "sudo" from deploy's stage (boot time dropped from 20-50s to 1-6s, see https://docs.travis-ci.com/user/ci-environment/ )

@bsideup bsideup requested review from kiview and rnorth June 28, 2017 20:32
@rnorth
Copy link
Copy Markdown
Member

rnorth commented Jul 1, 2017

Nice! LGTM so will merge.

@rnorth rnorth merged commit 255a1e3 into master Jul 1, 2017
@rnorth rnorth deleted the speed_up_travis branch July 1, 2017 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rnorth rnorth Awaiting requested review from rnorth

@kiview kiview Awaiting requested review from kiview

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

@bsideup @rnorth