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

Add yay specific args, drop conflicting args #615

Merged
merged 2 commits into from
Aug 5, 2018

Conversation

Morganamilo
Copy link
Contributor

No description provided.

@Morganamilo
Copy link
Contributor Author

Got any suggestions to replace --print or are you fine with just the short option?

@AladW
Copy link
Contributor

AladW commented Aug 2, 2018

I guess that would make it the only short option without corresponding long option. No idea is this of any importance. I'd consider --yayconfig instead of --currentconfig though, to emphasize the difference with --config.

@Morganamilo
Copy link
Contributor Author

I guess that would make it the only short option without corresponding long option. No idea is this of any importance.

It's very important for my OCD.

I'd consider --yayconfig instead of --currentconfig though, to emphasize the difference with --config.

We already have --defaultconfig, --currentconfig matches it very well.

@Jguer
Copy link
Owner

Jguer commented Aug 4, 2018

Only alternative of --print I can think right now is --puke information. Only the short option is not very satisfying, even if I've never used the long option

@Morganamilo
Copy link
Contributor Author

--show
--output
--out
--echo

Are all I can think of right now. --show is my pick, even if it is a little annoying that it does not start with p.

show news
show currentconfig
show stats
show completion

All make perfect sense to me (although the completion flag is actually complete). If there's no objection I'll update the PR and merge it when I get round to it.

Replace -P/--print with -P/--show
Replace --config with --currentconfig
@Jguer
Copy link
Owner

Jguer commented Aug 5, 2018

--show is okay I guess, just going to be a nice laugh when someone asks why -Sc is deleting their cache instead of printing completions

@Morganamilo
Copy link
Contributor Author

Eh, -S is probably the most well known pacman operation. If they confuse --show for -S they've only got themselves to blame.

@Morganamilo Morganamilo merged commit a5ae757 into Jguer:master Aug 5, 2018
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.

3 participants