Skip to content

Drop Support for [email protected] #698

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
Mar 2, 2025

Conversation

seanpdoyle
Copy link
Contributor

Related to #681
Follow-up to #682

Commit to requiring [email protected]

Remove [email protected], [email protected], [email protected], [email protected], and [email protected] from the CI matrix, along with any RUBY_VERSION < "3" conditionals.

The implementation changes include replace older syntaxes with newer variants. For example, utilize ... in place of *args, &block, rely on **-ing attribute Hash arguments, etc.

@seanpdoyle seanpdoyle force-pushed the drop-support-for-ruby-2 branch 5 times, most recently from 1035891 to 600f23f Compare November 4, 2024 13:17
@seanpdoyle seanpdoyle force-pushed the drop-support-for-ruby-2 branch 3 times, most recently from 251dbbc to e7a8848 Compare February 2, 2025 00:35
Commit to requiring `[email protected]`

Remove `[email protected]`, `[email protected]`, `[email protected]`, `[email protected]`, and `[email protected]`
from the CI matrix, along with any `RUBY_VERSION < "3"` conditionals.

The implementation changes include replace older syntaxes with newer
variants. For example, utilize `...` in place of `*args, &block`, rely
on `**`-ing attribute `Hash` arguments, etc.
@seanpdoyle seanpdoyle force-pushed the drop-support-for-ruby-2 branch from e7a8848 to 79f7cce Compare March 2, 2025 21:16
@seanpdoyle
Copy link
Contributor Author

@jorgemanrubia I've rebased off main, and I think this fixes the failing tests.

Does this changeset achieve the same outcome as https://github.com/hotwired/turbo-rails/tree/fix-tests? If not, maybe you can cherry-pick the changes necessary to pass CI.

@jorgemanrubia jorgemanrubia merged commit 3fcec46 into hotwired:main Mar 2, 2025
14 checks passed
@jorgemanrubia
Copy link
Member

Lovely, thanks @seanpdoyle! I had started to look into fixing the tests, I didn't realize you had this one open 🙏

@seanpdoyle seanpdoyle deleted the drop-support-for-ruby-2 branch March 2, 2025 21:26
@SleeplessByte
Copy link

FYI (for future readers), this being a patch version change broke our build unexpectedly (of course caught by CI, so impact minimal). This release increased the minimum Rails version to 7.1, which is not yet in Debian (stable).

If you are on Debian 12, pin to v2.0.12, or vendor it. In fact you can grab v2.0.13 and manually edit the .gemspec file, if you're stuck on Debian 12 for now, as Debian ruby is 3.1.2, and there is no actual requirement (in this repo's code) to run Rails 7.1+, yet. Trixie has just started the freeze, which means that Debian 13 will likely release with Rails 7.2.2.1.

For those thinking that Rails 6.1 / 7.0 are EOL, the Ruby Debian team is still backporting bugfixes, as is the promise for all things Ruby in Debian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants