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

short flags for --yaml-input and --yaml-output #211

Closed
wants to merge 1 commit into from
Closed

short flags for --yaml-input and --yaml-output #211

wants to merge 1 commit into from

Conversation

imarko
Copy link

@imarko imarko commented Mar 16, 2023

Add -y and -Y as short flags for --yaml-input and --yaml-output.

@imarko
Copy link
Author

imarko commented Mar 16, 2023

I've been using these in my local copy so that I can gojq -yY for easy YAML processing but always assumed that it wasn't in upstream because of (future) flag compatibility caution. But maybe that's not the case and I don't see a previous issue or PR so I figure it's worth a try.

@itchyny
Copy link
Owner

itchyny commented Mar 17, 2023

This could be useful for users switching these flags, but give me more time to decide. In my environment, I have a simple shell alias to enable these options and that's working well. Also, I'm afraid users might be confusing about the short flag capitals; especially for being the opposite against the raw string flags.

@itchyny
Copy link
Owner

itchyny commented Mar 23, 2023

I can't see any benefit at the risk of short flag collision in the future. Also these flags may confuse the users due to the reason in my previous comment.

@itchyny itchyny closed this Mar 23, 2023
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