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

@mikevoets mikevoets commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

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

geemus commented Apr 21, 2026

Copy link
Copy Markdown
Member

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

geemus commented Apr 21, 2026

Copy link
Copy Markdown
Member

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

geemus commented Apr 21, 2026

Copy link
Copy Markdown
Member

@mikevoets Thanks for the fix!

@geemus

geemus commented Apr 21, 2026

Copy link
Copy Markdown
Member

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