-
Notifications
You must be signed in to change notification settings - Fork 123
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
feat(add compact arrays): we can now have nice one line syntax for arrays #299
Conversation
I think it has good as it can gets for this specific use case, there is still the issue of structs inside array so maybe compact_arrays could be extended to compact_array_if_level_below(level) so only leaf structure are compact but nested ones aren't |
Either that, or define a char limit for "compact arrays" (could do the same with structs / maps, too, I guess) |
Here are 2 things that we'll need to follow-up with:
|
Issue has had no activity in the last 60 days and is going to be closed in 7 days if no further activity occurs |
comment |
Add compact arrays (#299) and pretty inline separators
The |
Add compact arrays (ron-rs#299) and pretty inline separators
This refer to my recent issue #298 where I mention I would love a compact array definition,
This PR allow to do so by using the
with_compact_arrays(true)
, the default is false ofc.