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

Use sudo: false for the Travis configuration and update Ruby versions #170

Merged
merged 2 commits into from
Jan 16, 2017

Conversation

aprescott
Copy link
Contributor

This enables the container-based architecture, which is generally better, and has the benefit of using a later version of bundler.

I believe this will help get #163 passing, which will then help address #151.

This enables the container-based architecture, which is generally
better, and has the benefit of using a later version of bundler.
@aprescott
Copy link
Contributor Author

In case anyone is wondering, this fix for the issue in #151 comes from rubygems/bundler#3558 (comment).

It used to be the case that RVM (and thus Travis) would let you use,
say, "2.0" to refer to the latest 2.0.x release. Patch versions now need
to be given explicitly. See github/rvm/rvm 3512.
@aprescott
Copy link
Contributor Author

The build on my first commit was failing because of rvm changes and "2.3" no longer being available for install. See rvm/rvm#3512.

If people would prefer I issue a separate PR with the .travis.yml changes, I can do that, or I can leave it in this PR.

@aprescott aprescott changed the title Use sudo: false for the Travis configuration Use sudo: false for the Travis configuration and fix version strings Jan 13, 2017
@aprescott aprescott changed the title Use sudo: false for the Travis configuration and fix version strings Use sudo: false for the Travis configuration and update Ruby versions Jan 13, 2017
@kschiess kschiess merged commit a80c077 into kschiess:master Jan 16, 2017
@kschiess
Copy link
Owner

Thank you!

@aprescott aprescott deleted the travis-no-sudo branch January 16, 2017 15:09
@aprescott aprescott mentioned this pull request Jan 16, 2017
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.

2 participants