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.

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

Use JSON if Available #1

Closed
sferik opened this issue Sep 16, 2011 · 7 comments · Fixed by #19
Closed

Use JSON if Available #1

sferik opened this issue Sep 16, 2011 · 7 comments · Fixed by #19

Comments

@sferik
Copy link
Member

sferik commented Sep 16, 2011

If JSON is available, use it instead of YAML (for perfs).

@laserlemon
Copy link

Out of curiosity, why use YAML in the first place?

@sferik
Copy link
Member Author

sferik commented Oct 2, 2011

@laserlemon I specifically designed gems to have no dependencies and JSON isn't part of stdlib in Ruby < 1.9.

@laserlemon
Copy link

Ahh, yeah I thought that might be the case once I looked over the gemspec more carefully. And when I saw that Faraday was nowhere to be found. Cool, I dig it. Is JSON much more performant?

@sferik
Copy link
Member Author

sferik commented Oct 2, 2011

Yes. It's smaller over the wire and should be faster to parse.

@sunnyrjuneja
Copy link

Is this is still an endeavor worth pursuing? Are people still using this gem?

@sferik
Copy link
Member Author

sferik commented Feb 5, 2014

This gem gets about 600 downloads per week, so yes, it is still being used. Are you interested in implementing this feature?

@sunnyrjuneja
Copy link

Yes! Anything I should know before starting? I can also take care of #16 at the same time.

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 a pull request may close this issue.

3 participants