quicli icon indicating copy to clipboard operation
quicli copied to clipboard

Add serde_json?

Open killercup opened this issue 7 years ago • 4 comments

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

killercup avatar Jan 22 '18 09:01 killercup

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

killercup avatar Jan 22 '18 09:01 killercup

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

moosingin3space avatar Jan 23 '18 18:01 moosingin3space

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?

killercup avatar Jan 24 '18 20:01 killercup

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.

acheronfail avatar Jan 31 '18 10:01 acheronfail