- Notifications
You must be signed in to change notification settings - Fork18
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looking solid. Just a few language and code nits and here and there.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
if outputFmt == jsonOutput { | ||
enc := json.NewEncoder(os.Stdout) | ||
// pretty print the json | ||
enc.SetIndent("", "\t") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Nice 💯
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
I pushed up the suggested changes and updated the PR description to reflect the new human-readable output that includes the image tag now. If any of the command usage copy looks off or reads awkward, lmk what text you think we should have in its place. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
3b2afa4
to30126a3
Compare
Uh oh!
There was an error while loading.Please reload this page.
What this does
Allows users to list imported images.
By default, a
coder images ls
will list every imported image for every organization a user belongs to.The default output is a human-readable table that describes the default resource amounts for each image :
Users can filter images by organization using the
--org
flag.Users may optionally choose to have the images output in a pretty-printed JSON format using
--output json
.