-
Notifications
You must be signed in to change notification settings - Fork 70
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
Travis: drop unsupported Rubies #82
base: master
Are you sure you want to change the base?
Conversation
- Also: Removes an old setting from Travis. See https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration
.travis.yml
Outdated
@@ -21,12 +18,11 @@ env: | |||
matrix: | |||
exclude: | |||
- env: HANDLER="ox" | |||
rvm: jruby-1.7 | |||
rvm: jruby-9.2.6.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you can safely use jruby-9
instead and travis will figure out what the current release is for you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not been successful letting Travis CI's rvm
pick a Ruby which does that for me.
In rvm
, the information does exist. https://github.com/rvm/rvm/blob/master/config/known#L19
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, for some reason feedjira doesn't seem to mind.
https://github.com/feedjira/feedjira/blob/master/.travis.yml#L9
Either way, dropping jruby-1.7 seems like a great idea! 💯
This PR was inspired by the build failure in #81.
It removes the no-longer-supported Ruby versions from the Travis CI matrix.