Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adds multi_json #794

Closed
wants to merge 1 commit into from
Closed

Adds multi_json #794

wants to merge 1 commit into from

Conversation

warmwaffles
Copy link

Allows others to choose their flavor of json parsing.

@warmwaffles
Add multi_json to dependency
2f2eedd 
Allows others to choose their flavor of json parsing.
@NuckChorris
Copy link

👎 multi_json is slowly being removed from other projects in favor of just using the built-in JSON gem, especially now that both OJ and Yajl have a compatible mode via Oj.mimic_json and require 'yajl/json_gem'. There's really no point to MultiJson any more

@warmwaffles
Copy link
Author

warmwaffles commented Aug 12, 2016 •
edited
Loading

https://github.com/ohler55/oj/blob/v2.17.1/lib/oj/mimic.rb

@warmwaffles
Copy link
Author

I get what you are saying though, jmespath/jmespath.rb#6 (comment)

@warmwaffles warmwaffles deleted the multi_json branch August 12, 2016 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@warmwaffles @NuckChorris