celluloid / reel

UNMAINTAINED: See celluloid/celluloid#779 - Celluloid::IO-powered web server
https://celluloid.io
MIT License
596 stars 87 forks source link

Travis: Update Ruby versions in CI matrix #243

Closed olleolleolle closed 7 years ago

olleolleolle commented 7 years ago

This PR updates Ruby versions to latest releases in their series.

olleolleolle commented 7 years ago

Proposal: Perhaps we could let go of 2.1?

olleolleolle commented 7 years ago

The current JRuby 1.7 build failure is about http_form's missing required_ruby_version in its .gemspec which does not preclude 1.9 Rubies from automatically using it.

It'd be nice if that gem had such metadata in its .gemspec.

olleolleolle commented 7 years ago

JRuby 1.7 is EOL'd, would it be more beneficial in the long run to hop over to JRuby 9.1.9.0?

ixti commented 7 years ago

@olleolleolle I'm sorry about that. Will fix form_data asap

ixti commented 7 years ago

@olleolleolle Can you please re-run tests. v1.0.3 of form data now fixed to play nice with jruby 1.7 and ruby 1.9+

olleolleolle commented 7 years ago

@ixti That fixed things here! Yes!

:hearts: :yellow_heart: :green_heart: :blue_heart: :purple_heart:

olleolleolle commented 7 years ago

@digitalextremist Is there anything more you'd like from this PR? Or, any change you'd like to see done to it?

kenichi commented 7 years ago

thank you - i think this is great as a specific PR, and i'm curious as to your thoughts about my other comments and what i've done to CI and deps in my own PR. i would be happy to revert/rebase from this!

.travis.yml Gemfile reel.gemspec

olleolleolle commented 7 years ago

@tarcieri I'm ready for code review on this one, now.

olleolleolle commented 7 years ago

@kenichi Oh, there we are, merged and good to go. Thank you for all of the input, it made this a lot better. Better is better!