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-skip] Add 6.0 Release Notes for Active Record #35883
[ci-skip] Add 6.0 Release Notes for Active Record #35883
Conversation
I think #35844 is missing. |
e3ac49d
to
e59b40e
Compare
@prathamesh-sonpatki thank you very much for the thorough review! I've updated my PR to reflect the changes. @simi Question: is the noteworthy change in #35844 that Rails 6 is compatible with |
e59b40e
to
ba57b76
Compare
@alexcameron89 it is not making AR compatible with 1.4, but actually it makes AR incompatible with |
@simi That makes sense, thank you for the additional context. I added it in 029da21 |
@alexcameron89 Can you squash the commits into one? |
029da21
to
4416158
Compare
@prathamesh-sonpatki done 👍 |
Summary
This PR adds Rails 6.0 Release Notes to Active Record based off of the Changelog.
Thoughts related to this PR
Not knowing what specifically should be notable changes, I opted for including more than less. I'm glad to remove any that should not be included.
I also thought that it would be nice to categorize the notable changes with headers based off of major features, version bumps, or fixes. While I haven't implemented that change, I'm very glad to do so.