-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix system df
issues with -f
and -v
#16234
Fix system df
issues with -f
and -v
#16234
Conversation
7da7d90
to
6c0ca79
Compare
I think something got screwed up when I had to fix some validate issues. Will figure out how to fix that. |
Could you update the man page. |
Fixed the issue of `--format` and `--verbose` flags being allowed in combination with one another. Implemented functionality for `--format json` or `--format '{{ json }}' `. Implemented command-completion help for `--format`. Fixes: containers#16204 Signed-off-by: Jake Correnti <[email protected]>
6c0ca79
to
e5032a8
Compare
looks like this is affected by #16076 |
Flakes finally green after multiple restarts. Tests LGTM. @containers/podman-maintainers PTAL. |
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.
LGTM
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jakecorrenti, vrothberg The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Nice work! |
Fixed the issue of
--format
and--verbose
flags being allowed in combination with one another.Implemented functionality for
--format json
or--format '{{ json }}'
.Implemented command-completion help for
--format
.Fixes: #16204
Signed-off-by: Jake Correnti [email protected]
Does this PR introduce a user-facing change?