apokalipto / devise_saml_authenticatable

Devise SAML 2.0 authentication strategy
MIT License
294 stars 153 forks source link

Add Ruby 3.0 and 3.1 to CI #211

Closed petergoldstein closed 2 years ago

petergoldstein commented 2 years ago

This PR adds Ruby 3.0 and 3.1 to CI and updates the CI Ruby/Rails matrix accordingly. Changes outside of the workflow definition include:

This PR doesn't include Rails 7, as getting that working properly requires a bit more work with views and UI behavior.

petergoldstein commented 2 years ago

Great, thanks. I'm working on getting Rails 7 in now. It's a little involved because the ruby-saml-idp gem is unmaintained and is incompatible with zeitwerk class loading. I'm finishing up an updated, maintained version of that gem that can be used by devise_saml_authenticatable going forward.

petergoldstein commented 2 years ago

@adamstegman On a related note, is there a reason the support matrix is so wide? It goes back to Rails 4.0 and Ruby 2.0.

Ruby 2.5 was EOLed earlier this year. Limiting it to Ruby 2.5+ or Ruby 2.6+ would simplify things.

Similarly, Rails 5.1 was EOLed over 2 years ago. Limiting it to Rails 5.2+ would also simplify things.

adamstegman commented 2 years ago

Good point! We can probably drop support for versions that are no longer supported upstream without having to bump a major version.

Thanks for digging into Rails 7 too!

petergoldstein commented 2 years ago

@adamstegman If you're good with those changes I'll put up a PR to trim CI to Ruby 2.6+ and Rails 5.2+