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

CI against Ruby 3.0.0 #1190

Merged
merged 1 commit into from
Dec 28, 2020
Merged

CI against Ruby 3.0.0 #1190

merged 1 commit into from
Dec 28, 2020

Conversation

yahonda
Copy link
Contributor

@yahonda yahonda commented Dec 25, 2020

This pull request runs CI against Ruby 3.0.0.
Rails 6.0.3 or higher is required to support Ruby 3 keyword arguments behavior then excluded the matrix between Ruby 3.0.0 and Rails 5.2.x.

This version fixes warnings when used with Ruby 2.7.

Keyword arguments are separated from other arguments.

Rails 6.0.3 or higher is required to support Ruby 3 keyword arguments
behavior.

* Rails 6.0.3 has been released!
https://weblog.rubyonrails.org/2020/5/6/Rails-6-0-3-has-been-released/
> This version fixes warnings when used with Ruby 2.7.

* Ruby 3.0.0 Released
https://www.ruby-lang.org/en/news/2020/12/25/ruby-3-0-0-released/
> Keyword arguments are separated from other arguments.
Copy link
Contributor

@deivid-rodriguez deivid-rodriguez left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Should we consider dropping Rails 5.2 support?

@yahonda
Copy link
Contributor Author

yahonda commented Dec 28, 2020

Thanks for the review. In my opinion, Dropping Rails 5.2 support is still needed now.

@deivid-rodriguez
Copy link
Contributor

@yahonda Do you mean that "Rails 5.2 is still needed" or that "dropping Rails 5.2 support is still needed"? Couldn't understand if you're in favor or against the change 😅.

@yahonda
Copy link
Contributor Author

yahonda commented Dec 28, 2020

Thanks for the confirmation. I meant to say "Rails 5.2 is still needed" .

@deivid-rodriguez
Copy link
Contributor

Ok, I don't have a special interest in dropping 5.2 support right now so let's leave it for now.

@deivid-rodriguez
Copy link
Contributor

Thanks for this!

@deivid-rodriguez deivid-rodriguez merged commit fa8f1bb into activerecord-hackery:master Dec 28, 2020
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.

2 participants