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
CI: Travis maintenance: add JRuby 9.2; fix JRuby 9.1 build; configure Rubinius #921
Conversation
https://travis-ci.org/github/savonrb/savon/jobs/580391898 was the failed job that I noted. |
@pcai Would you be comfortable with cutting out non-maintained Ruby versions? |
Thank you! Yes, I think we should cut unmaintained ruby. |
@pcai I'll continue doing that in this PR, if you 👍 it. In other words: Do you want this PR to remove the unmaintained Ruby versions from CI? In addition, would you want me to set the |
Nice work! |
@olleolleolle thanks. Yes if you open up another PR to update the gemspec then I will merge that too. IMO new releases should only be compatible with and tested for currently maintained versions of ruby. |
… Rubinius (#921) * CI: Extend matrix w/ JRuby 9.2 * CI: run Rubinius in trusty * CI: JRuby 9.2 JAVA_OPTS with --add-opens decls See jruby/jruby#4834 * CI: Use "name" to identify specific matrix jobs * CI: Drop unmaintained Ruby versions - see https://www.ruby-lang.org/en/downloads/branches/
This is a CI change to use latest release of JRuby 9.1 (which is a 2.3, now out of support, see Ruby's support policy).
Did you add tests for your changes?
No, no new tests were added - this is a CI-only change.
Summary of changes
The attempt is: try to make the JRuby build pass.
allow_failures
allow_failures
Closes #910 by including its fix here.