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

Print out namespaces list as json by default #541

Merged
merged 3 commits into from
Sep 11, 2019

Conversation

gabbifish
Copy link
Contributor

Make json output default for listing namespaces as well (and update documentation)!

@EverlastingBugstopper
Copy link
Contributor

Do we want to default to pretty print with the table and then print json if we pass a --json flag? I feel like that's the usual pattern for tools that output json. Not a hill I'll die on though

@gabbifish
Copy link
Contributor Author

@EverlastingBugstopper I think we can get away without offering pretty-printing right now--there are plenty of other commandline tools one can just pipe the wrangler output to in order to pretty-print it :) (I include an example of this in the docs)

docs/content/kv_commands.md Outdated Show resolved Hide resolved
@gabbifish gabbifish added this to the Add KV Subcommands milestone Sep 11, 2019
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gabbifish gabbifish merged commit 69ac09a into feat-kv-commands Sep 11, 2019
@delete-merged-branch delete-merged-branch bot deleted the gabbi/list-namespaces-json branch September 11, 2019 22:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants