devise icon indicating copy to clipboard operation
devise copied to clipboard

Make test suite work

Open gregmolnar opened this issue 9 months ago • 2 comments

The tests on main are not passing with Rails 7.1 onwards. At the same time, since Rails is supported 7.2 onwards and Ruby 3.1 is EOL in 2 weeks, we can drop support for older versions.

gregmolnar avatar Mar 16 '25 16:03 gregmolnar

@gregmolnar can the tests and rails version support split into two PRs?

fthobe avatar Mar 21 '25 16:03 fthobe

No, because the test don't pass and it doesn't make sense to make them pass for versions of Ruby and Rails that Devise shouldn't support anymore.

gregmolnar avatar Mar 21 '25 19:03 gregmolnar

@carlosantoniodasilva These still pass locally with ruby 3.2 and 3.3. If this gets merged, I can look into the issues with 3.4

gregmolnar avatar Oct 16 '25 15:10 gregmolnar

@gregmolnar thanks for this.

I was originally considering only dropping support to Rails < 6 in Devise, regardless of Rails currently supported versions. But that's been a while, and I've already dropped support for Rails < 7 and Ruby < 2.7 on other heartcombo/* libs (was gonna get to Devise next/soon), so I think we should stick to that same level of support for Devise here, if you don't mind.

What's in main is already cutting down multiple versions (currently released version supports all the way back to 4.1), so let's at least leave the last 2 majors on the table; cut down v4/5/6, and continue supporting v7.0+ for the time being.

This is responders setup, for reference: https://github.com/heartcombo/responders/commit/818ec078bcb37d7729db5fd34883e08ae031f038

carlosantoniodasilva avatar Oct 16 '25 20:10 carlosantoniodasilva

@carlosantoniodasilva I made the test matrix the same and added back Rails 7.0 support. Can you approve the workflow to see if the whole matrix succeeds? Thanks!

gregmolnar avatar Oct 16 '25 21:10 gregmolnar

@carlosantoniodasilva I fixed the issues, it passes on my fork now: https://github.com/gregmolnar/devise/actions/runs/18576202141

gregmolnar avatar Oct 16 '25 22:10 gregmolnar

Thank you! ❤️💜💛💚💙

carlosantoniodasilva avatar Oct 21 '25 19:10 carlosantoniodasilva