Skip to content

Uses new containerized Travis build system #163

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

Merged

Conversation

migmruiz
Copy link
Contributor

@rorygraves rorygraves merged commit 654054b into scoverage:master Jun 3, 2016
@rorygraves
Copy link
Contributor

Based on the fact travis actually passes with this I'm going to merge.

@gslowikowski
Copy link
Member

+sudo: false can be removed. False is default value.

This file is very similar to the one in my Java/Scala/Maven test projects. I have no sudo setting.

From Travis docs (above link):

If you see This job is running on container-based infrastructure in your build log it means you are already running builds on our new container-based infrastructure.

If you don’t, to use the new infrastructure add the following line to your .travis.yml:

sudo: false

Check Travis logs. There is This job is running on container-based infrastructure line. This one one is from latest build.

@rorygraves
Copy link
Contributor

Whoops - looks like I was a bit hasty on that merge.

@gslowikowski
Copy link
Member

No, it's OK, removing sudo option is just small improvement you can commit separately.

@rorygraves
Copy link
Contributor

Yeah I'll drop that in today

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

Successfully merging this pull request may close these issues.

3 participants