Skip to content

Support Rails 7.2 #255

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

Merged
merged 1 commit into from
Dec 18, 2024
Merged

Conversation

olivier-thatch
Copy link
Contributor

Support Rails 7.2

Rails 7.2 was released a few weeks ago. This PR upgrades minitest-rails accordingly.

Note that Rails 7.2 bumped the minimum Ruby version from 2.7 to 3.1, so I've done the same here.

@olivier-thatch
Copy link
Contributor Author

@blowmage Any chance this could be released soon?

@zenspider
Copy link
Collaborator

@blowmage nudge

@n-rodriguez
Copy link

Hi there! Any news? ping @blowmage

@bquorning
Copy link
Contributor

Based off of this branch, I opened #258 to add support for Rails 8.0 too.

Copy link
Collaborator

@zenspider zenspider left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. I'mma step in and try to maintain this for a while... As such I'm still catching up... I'd like to request some changes to strip this down to JUST supporting rails 7.2. The rest of this PR should be broken out to a separate commit and PR please.

This was referenced Nov 27, 2024
@zenspider
Copy link
Collaborator

Also I understand if it's been too long... I'm happy to make the changes requested.

@olivier-thatch
Copy link
Contributor Author

@zenspider Made the requested changes.

@olivier-thatch olivier-thatch force-pushed the olivier-rails-7-2 branch 2 times, most recently from 8a91008 to 0ea44aa Compare November 28, 2024 06:47
@n-rodriguez
Copy link

Hi there! Any news?

@zenspider
Copy link
Collaborator

I'm on it

@zenspider
Copy link
Collaborator

All the 7.2 tests passed and all the 7.1 tests failed, presumably because they have too old of a ruby version? I am trying to make sense of this but since the 7.2 tests are passing I'mma consider this good and I'll fix the CI out of band.

@zenspider zenspider merged commit 5014f29 into minitest:master Dec 18, 2024
3 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants