Add optional serde feature (enabled by default)#413
Add optional serde feature (enabled by default)#413LucaCappelletti94 wants to merge 5 commits intoBurntSushi:masterfrom
serde feature (enabled by default)#413Conversation
|
Thanks! It might be some time before I have the bandwidth to review and get this merged. |
I am fixing the last failing CI, do let me know if I can facilitate in any way the PR. |
|
While I am at it, I will also do a round of clippy code-smell fixes. Note that the changes in the code are strictly limited to adding serde feature gates, nothing fancier than that. I have added also a run in the CI for the no-default-feature case, so everything is looking a-okay right now. |
|
Assuming that the current PR should be more or less merged as-is (given the triviality of it) I will get started with the |
Makes the
serdedependency optional (enabled by default) by gating all serde-related functionality behind aserdefeature flag.This PR will make it possible to use
csvin other crates which offer theserdefeature flag, and makes the groundwork for adding an optionalstdfeature flag which will be necessary to usecsvwith other crates which work with solelyalloc.Fixes #412, and replaces the old closed PR #175