Add some obvious short options #44
Conversation
| listFlags = []cli.Flag{ | ||
| cli.BoolFlag{ | ||
| Name: "quiet", | ||
| Name: "quiet, q", |
There was a problem hiding this comment.
Thought we were doing 'omit' here to kill the column heading and 'quiet' to show just the id column. fwiw, docker ps --quiet causes it to display only the numeric ids.
There was a problem hiding this comment.
That sounds good to me. Basically you want to add an option though correct, since there will still be a quiet option. You just want to change the meaning.
There was a problem hiding this comment.
Yes, I'd like to add an "omit" option here that would keep the table headers from printing when used. "quiet" would be used to show just the image id's, much like docker does. My main confusion here is I'd thought Nalin had already done this, but perhaps it wasn't merged?
There was a problem hiding this comment.
Well lets do it here or in a separate pull request.
|
@TomSweeneyRedHat I added a --noheading switch and changed quiet to work the way you wanted. |
Add `noheading` option to eliminate headings
|
LGTM. |
Add `noheading` option to eliminate headings Closes: #44 Approved by: rhatdan
|
☀️ Test successful - status-redhatci |
4 participants
I chose to only add traditional ones like
--quiet, -q
and
--user -u
And for some which could be specified multiple times I added
-v, -e, -l