Skip to content
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

Drop Support for [email protected] #698

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
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.

2 participants