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

Rails 6 #1403

Merged
merged 5 commits into from
Sep 13, 2019
Merged

Rails 6 #1403

merged 5 commits into from
Sep 13, 2019

Conversation

no-reply
Copy link
Contributor

Add support for Rails 6. Drop support for 5.1.

Tom Johnson added 5 commits September 10, 2019 12:39
Rails 5.1 support is ended. ActiveFedora 14 will drop support.
This method is removed in ActiveModel 6.0, and is private anyway. Avoid calling
it for upgradability.
We used to manually set `ActiveModel::Dirty` internals to track "previous
changes". In Rails 6, these internals are completely rewritten and much more
efficient. We can rely on the provided implementation instead of rolling our
own.
Rails 6 is here. 🎉
@coveralls
Copy link

coveralls commented Sep 10, 2019

Coverage Status

Coverage decreased (-0.01%) to 89.575% when pulling 8553227 on rails-6 into 2b75d5e on master.

Copy link
Member

@cjcolvar cjcolvar left a comment

Choose a reason for hiding this comment

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

This looks good to me.
For what it is worth, this probably means a new major release when we just did one less than a month ago.

@no-reply
Copy link
Contributor Author

yeah, i was assuming we'd release 14.0.0 immediately.

@no-reply no-reply merged commit 16e1f5e into master Sep 13, 2019
@cjcolvar cjcolvar deleted the rails-6 branch September 13, 2019 19:30
@no-reply
Copy link
Contributor Author

new plan, it's made backwards compatible in #1407

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.

3 participants