Skip to content

Add f.ObjectSeparator flag#7

Open
rm-hull wants to merge 2 commits intoTylerBrock:masterfrom
rm-hull:master
Open

Add f.ObjectSeparator flag#7
rm-hull wants to merge 2 commits intoTylerBrock:masterfrom
rm-hull:master

Conversation

@rm-hull
Copy link
Copy Markdown

@rm-hull rm-hull commented Sep 23, 2022

With this flag set to true (which is the default) and f.Indent = 0, then the behaviour is as previous, eg.

{ "a": 5, "b": [ 1, 2, 3 ], "c": { "d": "e" } }

When f.ObjectSeparator = false and f.Indent = 0, it would render the object as:

{"a":5,"b":[1,2,3],"c":{"d":"e"}}

@rm-hull
Copy link
Copy Markdown
Author

rm-hull commented Feb 11, 2023

@TylerBrock any appetite for merging this?

@TylerBrock
Copy link
Copy Markdown
Owner

Yes, sorry for the immense delay. Can you explain the use case a little more? Is the idea simply to have a more compact representation?

@rm-hull
Copy link
Copy Markdown
Author

rm-hull commented Jul 24, 2023

to have a more compact representation?

Exactly, especially when dealing with JSONL files

@rm-hull
Copy link
Copy Markdown
Author

rm-hull commented Aug 15, 2024

Could this be merged @TylerBrock ?

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