Skip to content
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

cli: Add table output for policy_status sub-command #888

Merged
merged 2 commits into from
Sep 7, 2023

Conversation

JAORMX
Copy link
Contributor

@JAORMX JAORMX commented Sep 7, 2023

This adds table output for the policy_status subcommand, making the
output a little more friendly. It also adds colors and emojis to the status
column as to reflect the reader's eyes towards that.

The --all flag in the policy_status list was changed to --detailed to
reflect better what the flag does.

Closes #876

This adds table output for the `policy_status` subcommand, making the
output a little more friendly. It also adds colors and emojis to the status
column as to reflect the reader's eyes towards that.

The `--all` flag in the `policy_status list` was changed to `--detailed` to
reflect better what the flag does.
@JAORMX JAORMX requested a review from jhrozek September 7, 2023 12:01
jhrozek
jhrozek previously approved these changes Sep 7, 2023
Copy link
Contributor

@jhrozek jhrozek left a comment

Choose a reason for hiding this comment

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

I don't see any glaring issues and the tables seem to look nice!
/shipit

@JAORMX JAORMX merged commit e56ea82 into main Sep 7, 2023
12 checks passed
@JAORMX JAORMX deleted the table-policy-status branch September 7, 2023 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Format policy CLI output to table format
4 participants