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

stringify defaulting to JSON #36

Closed
charlierudolph opened this issue May 6, 2015 · 2 comments · Fixed by #37
Closed

stringify defaulting to JSON #36

charlierudolph opened this issue May 6, 2015 · 2 comments · Fixed by #37

Comments

@charlierudolph
Copy link

Why does the stringify method default to JSON if the indentation level is falsey?
This in unexpected and seems unnecessary as you could create valid minified cson

@jkrems
Copy link
Contributor

jkrems commented May 6, 2015

Hi! Thanks for bringing this up!

I don't think there's a specific reason. It was just the easiest way to implement it at the time. And when everything is formatted as one line, human readable and -editable becomes pretty abstract. That being said: if someone would want to take this on and create a more minimal single-line representation I don't think anything would speak against merging it.

@johan
Copy link
Contributor

johan commented May 6, 2015

Yeah, it was mostly incidental – no special intent besides ease of implementation and nobody having skin in the game about what non-indented cson should look like, as we mainly have been interested in cson as a human-consumption-friendly tidy subset of yaml. Aspirations, discussions and pull requests cordially invited, especially at a lovely civil level like this. :-)

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