-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
Cool! It's not easy for me to tell from reading this, but,
|
@theinventor , hi!
Well, I am not sure about the last part but if there is something wrong or missing, please, tell me and I will make appropriate fixes/changes. |
aa07917
to
882020d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding merging this for https://repairtechsolutions.atlassian.net/browse/RS-10919
Looks like this removes mimemagic and so rails 6.0.3.6 is sufficient.
So do we need to have QA, or perhaps just coordinate with shipping and testing in production if rollback required.
Upgrade to the latest version of Rails and dependencies.