Skip to content

Conversation

@JonRowe
Copy link
Member

Rails 7 will be supported in rspec-rails 6 via main, this just tracks Ruby 3.1 support into the upcoming 5.1.x release.

@pirjpirj mentioned this pull request Jan 20, 2022
21 tasks
@pirjpirjforce-pushed the ruby-31-5-1-maintenance branch 2 times, most recently from 34c6391 to 3cc2457CompareJanuary 21, 2022 19:05
Add additional logic to the ActionMailer argument parsing to accomodate for differences under Ruby 3.1/Rails 6.1 Co-authored-by: Fabio Napoleoni <[email protected]> Co-authored-by: Phil Pirozhkov <[email protected]>
@pirjpirjforce-pushed the ruby-31-5-1-maintenance branch from 3cc2457 to a687a8eCompareJanuary 21, 2022 19:08
@pirj
Copy link
Member

pirj commented Jan 21, 2022

Note to self - don't call branches ending with *-maintenance to prevent double the builds 😄

@pirj
Copy link
Member

pirj commented Jan 21, 2022

@JonRowe Green. Does it look good to you?
Fixes for Rails 7 and Ruby 3.1 got interleaved. I extracted the part that I could remember was specifically for Rails 7.

@pirj
Copy link
Member

pirj commented Jan 22, 2022

I'll go ahead and merge this.

@pirjpirj merged commit fae1360 into 5-1-maintenanceJan 22, 2022
@pirjpirj deleted the ruby-31-5-1-maintenance branch January 22, 2022 20:26
@JonRowe
Copy link
MemberAuthor

@pirj I'm going to be unpicking this a bit, it includes changes not needed, I was working on this on my machine but just hadn't responded.

@pirj
Copy link
Member

pirj commented Jan 23, 2022

Please let me know what you plan to remove from 5-1-maintenance. I may work on making the same changes on 5-0-maintenance in parallel.

Sign up for freeto 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.

3 participants

@JonRowe@pirj@petergoldstein