-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Shall it contain adapters for Display and FromStr? #3
Comments
Thanks for the idea! I would prefer not to include these. Maybe there could be a separate crate that provides such helpers for implementing miniserde::Serialize and miniserde::Deserialize, like what we have for Serde with crates like |
Shall it be mini-crates for each adapter and helper or one general big "miniserde toolbox" enhancement crate? Shall some documentation example (and maybe the |
If it were up to me, I would prefer one consolidated toolbox crate. But when adding any helpers that take a noticeable amount of time to compile or that pull in any additional dependencies, I would put those behind an off-by-default Cargo cfg. 😸 |
I posted a request for implementation in dtolnay/request-for-implementation#11. |
Like these:
Combined with strum it should provide easy miniserde for enums.
The text was updated successfully, but these errors were encountered: