Skip to content

Fix deprecation warning - #104

Merged
jarednorman merged 2 commits into
solidusio-contrib:masterfrom
folioco:deprecation
Apr 9, 2026
Merged

Fix deprecation warning#104
jarednorman merged 2 commits into
solidusio-contrib:masterfrom
folioco:deprecation

Conversation

@BrentWheeldon

Copy link
Copy Markdown
Contributor

MIME::Type now expects a hash of arguments for its constructor.

MIME::Type now expects a hash of arguments for its constructor.
@jarednorman

Copy link
Copy Markdown
Member

Is this backwards compatible or do we need to specify a version constraint on the mime-types gem?

@BrentWheeldon

Copy link
Copy Markdown
Contributor Author

It appears that the hash initialization has been supported since this PR which shipped as 2.0 in October 2013. I'm happy to add a > 2 constraint?

@jarednorman

Copy link
Copy Markdown
Member

Yes please!

@BrentWheeldon

Copy link
Copy Markdown
Contributor Author

Done!

@jarednorman

Copy link
Copy Markdown
Member

Sorry, I think ">= 2" is what we need, since 2 is just fine.

Since we now use the hash-based initializer we need at least v2.
@BrentWheeldon

Copy link
Copy Markdown
Contributor Author

🤦🏻 Indeed... Fixed!

@jarednorman
jarednorman merged commit c017a24 into solidusio-contrib:master Apr 9, 2026
1 check passed
@BrentWheeldon
BrentWheeldon deleted the deprecation branch April 9, 2026 20:17
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