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

Adding the --output table directive to public-ports alias #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chaunceyt
Copy link

No description provided.

@kyleknap
Copy link
Contributor

Thanks for the contribution. However, I think I would prefer keeping the --output parameter out of the alias. The main reason being that if it is hard coded into the alias, you cannot override the output format provided by --output or in the .aws/config file. With the way it is now, it will just be displayed in the format the user needs/prefers.

Could you elaborate on your reasoning for adding --output table to the alias?

@chaunceyt
Copy link
Author

Hi,
I added the directive "for consistency" because, I noticed that currently all of the describe aliases use --output table directive.
Thanks.

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.

2 participants