Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Add serde_json? #14

Open
killercup opened this issue Jan 22, 2018 · 4 comments
Open

Add serde_json? #14

killercup opened this issue Jan 22, 2018 · 4 comments

Comments

@killercup
Copy link
Owner

We already have serde and serde-derive, it might make sense to be able to read/write JSON as well.

@killercup
Copy link
Owner Author

I tried re-exporting the json! macro using the json_internal! macro but that could not be found for some reason (even though serde-json clearly macro_exports it).

@moosingin3space
Copy link

Use case: something like Docker CLI where you take JSON-formatted input or produce JSON-formatted output.

@killercup
Copy link
Owner Author

Let's ask a different question: What can we add in quicli that would make a typical CLI thing that involves JSON more comfortable/easy so it is superior to just extern crate serde_json?

@acheronfail
Copy link

Hmm I'm not sure supporting serde_json is the best idea. Including serde and serde_derive I think is good, but what if the cli app just needs serde_yaml or toml?
IMO probably better to leave including different serde_* implementations to each cli app.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants