-
Notifications
You must be signed in to change notification settings - Fork 3
feat: support flattened items and comma seperated arguments #49
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
Conversation
The tokens in `#[serde(tokens)]` are split by `,` and handled individually in order to allow attributes like `#[serde(default, flatten)]`
as in `#[toml_example(default = "\"hello!\", he shouted", require)]` or `#[serde(flatten, default)]`
|
@yanganto This PR should be ready for review now
|
|
Or rather, recursive nesting does not work regardless of the |
yanganto
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thank you. ❤️
Yeh, I think deep nesting and flatten are not the feature people using at the same time now. We can keep this in a separate issue. |
The output:
Notice the extra line between tuple and number: Should this be or would removing this be the better choice?close #48
To-do: