-
Notifications
You must be signed in to change notification settings - Fork 900
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 5.2 Upgrade #17338
Comments
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@juliancheal is this still an open issue? If not, can you close it. |
@JPrause Yes it's still open. |
Updating Cops ManageIQ/guides#338 |
@jrafanie can you look into this issue |
@chessbyte @jrafanie I think the last few issues from stopping us updating to 5.2 is ManageIQ/activerecord-virtual_attributes#39 |
@juliancheal, @jrafanie Is this done? |
No, I believe the only thing left was in activerecord virtual attributes. @kbrock do you know where we stand there with 5.2 support? Are we ready to try to get master on that? |
Sigh, I've been dragging. Think it requires a pre 5.1 version, a 5.2 version, and a 6.0 version. I need to get back to this |
Ok, I got that one pesky test passing for 5.2 ==> ManageIQ/activerecord-virtual_attributes#62 Lets see how those cross repo tests work, then we'll continue on the miq issues |
Addressed via #20032 |
This issue tracks progress for ManageIQ running Rails 5.2, released April 9, 2018
Rails 5.2 Release Notes
5-1-stable commit list
Known dependencies to be updated:
ManageIQ/manageiq-gems-pending#440
ManageIQ/manageiq-api-client#87
ManageIQ/ovirt_metrics#27
ManageIQ/vmware_web_service#59
ManageIQ/manageiq-providers-openstack#485
ManageIQ/manageiq-ui-classic#5824
ManageIQ/activerecord-virtual_attributes#39
ManageIQ changes
#18449
Release plan
The text was updated successfully, but these errors were encountered: