Skip to content

Make multi_json compatible with json 3.x - #67

Merged
byroot merged 1 commit into
sferik:mainfrom
byroot:json-3.x
Aug 12, 2026
Merged

Make multi_json compatible with json 3.x#67
byroot merged 1 commit into
sferik:mainfrom
byroot:json-3.x

Conversation

@byroot

@byroot byroot commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Ref: https://github.com/ruby/json/releases/tag/v3.0.0.rc1

Unknown keyword arguments will raise ArgumentError.

  • quirks_mode: no longer existing since json 2.0 (~2011).
  • create_additions no longer existing in json 3.0.

NB: rubocop prevent installing the RC, so I only did some local testing after messing with the Gemfile.

Comment thread lib/multi_json/adapters/json_gem.rb Outdated
@sferik

sferik commented Aug 12, 2026

Copy link
Copy Markdown
Owner

To handle RuboCop preventing json 3 from being installed, what would you think about moving the linters into a group :lint and adding a test matrix job that runs with bundle config set --local without lint and an unpinned json gem? Happy to do that in a separate PR if you'd prefer.

Ref: https://github.com/ruby/json/releases/tag/v3.0.0.rc1

Unknown keyword arguments will raise `ArgumentError`.

- `quirks_mode:` no longer existing since json 2.0 (~2011).
- `create_additions` no longer existing in json 3.0.
@byroot

byroot commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

what would you think about moving the linters into a group :lint

I don't thin that would be enough, as the constraints would still apply.

I did submit a PR to rubocop which was merged, hopefully it's released soon.

@byroot
byroot merged commit 7d14b25 into sferik:main Aug 12, 2026
25 checks passed
@byroot
byroot deleted the json-3.x branch August 12, 2026 22:23
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