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.
Development

No branches or pull requests

3 participants