Skip to content

Replace deprecated MultiJson.encode/decode with dump/load#24

Merged
geemus merged 1 commit into
fog:masterfrom
mikevoets:replace-deprecated-multijson-aliases
Apr 21, 2026
Merged

Replace deprecated MultiJson.encode/decode with dump/load#24
geemus merged 1 commit into
fog:masterfrom
mikevoets:replace-deprecated-multijson-aliases

Conversation

@mikevoets
Copy link
Copy Markdown
Contributor

@mikevoets mikevoets commented Apr 21, 2026

Summary

MultiJson.encode and MultiJson.decode are deprecated aliases and are slated for removal in MultiJson 2.0. Each call emits a deprecation warning via Kernel#warn:

MultiJson.decode is deprecated and will be removed in v2.0. Use MultiJson.load instead.
MultiJson.encode is deprecated and will be removed in v2.0. Use MultiJson.dump instead.

This PR swaps the internal calls to the canonical MultiJson.dump / MultiJson.load methods. Behavior is unchanged: dump/load are the real implementations that encode/decode alias to.

MultiJson.encode and MultiJson.decode are deprecated aliases and
scheduled for removal in MultiJson 2.0. Call the canonical MultiJson.dump
and MultiJson.load methods so downstream users do not see deprecation
warnings (which can escalate to errors when Ruby's warning handler is
configured to raise, e.g. via Bugsnag).
@geemus
Copy link
Copy Markdown
Member

geemus commented Apr 21, 2026

Just reactivated CI (it had stopped due to inactivity). Going to close/reopen this PR to hopefully trigger that.

@geemus geemus closed this Apr 21, 2026
@geemus geemus reopened this Apr 21, 2026
@geemus
Copy link
Copy Markdown
Member

geemus commented Apr 21, 2026

Trying again (did broader fixes to the tests/CI setup).

@geemus geemus closed this Apr 21, 2026
@geemus geemus reopened this Apr 21, 2026
@geemus geemus merged commit e44588d into fog:master Apr 21, 2026
4 of 9 checks passed
@geemus
Copy link
Copy Markdown
Member

geemus commented Apr 21, 2026

@mikevoets Thanks for the fix!

@geemus
Copy link
Copy Markdown
Member

geemus commented Apr 21, 2026

released in v1.6.0

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