Skip to content

Conversation

@seanpdoyle
Copy link
Contributor

@seanpdoyle seanpdoyle commented Sep 18, 2024

Closes #681

First, remove ruby@3.0 syntax including ... arguments and end-less
method definitions.

Next, add ruby@2.6 and ruby@2.7 to the CI matrix along with
rails@7.1.

@jorgemanrubia
Copy link
Member

Thanks @seanpdoyle. It seems that 2.X tests are still failing.

Closes [hotwired#681][] First, remove `ruby@3.0` syntax including `...` arguments and end-less method definitions. Next, add `ruby@2.6` and `ruby@2.7` to the CI matrix along with `rails@7.1`. [hotwired#681]: hotwired#681
@seanpdoyle
Copy link
Contributor Author

@jorgemanrubia I've restored CI to pass for ruby@2.6.x through ruby@3.3.x.

There's a few 2.x specific conditionals, but they can be removed once the final minor release supporting 2.x is cut.

Copy link
Member

@jorgemanrubia jorgemanrubia left a comment

Choose a reason for hiding this comment

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

Thanks @seanpdoyle!

@jorgemanrubia jorgemanrubia merged commit ca2c5f4 into hotwired:main Nov 4, 2024
22 checks passed
@seanpdoyle seanpdoyle deleted the issue-861 branch December 23, 2024 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants