Skip to content

Replace verbose option for "[A]ll - yes to all" when installing packages #2848

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

Closed
lemonlambda opened this issue Oct 6, 2022 · 0 comments · Fixed by #2849
Closed

Replace verbose option for "[A]ll - yes to all" when installing packages #2848

lemonlambda opened this issue Oct 6, 2022 · 0 comments · Fixed by #2849
Milestone

Comments

@lemonlambda
Copy link
Contributor

lemonlambda commented Oct 6, 2022

Is Your Feature Request Related To A Problem? Please describe.

No, it's just unnecessary info/difference from other package managers.

Describe The Solution. Why is it needed?

Remove the - yes to all part of the [A]ll prompt. It's a difference from all other modern package managers, and I think you should assume the end user knows what [A]ll means and if the they don't they can look it up easily.

Additional Context.

N/A

Related Issues

N/A

@pauby pauby added this to the 2.5.0 milestone May 12, 2025
gep13 pushed a commit to lemonlambda/choco that referenced this issue May 29, 2025
After some discussion with the Chocolatey team, it has been decided that
the prompt option for "All - yes to all" is rather verbose, and the need
to include "yes to all" is unneeded, and not in line with how other
package managers do it when prompted with similar choices.

This commit removes the "All - yes to all" and replaces it with simply
"All scripts", which is more succinct.  In addition, the unit/pester
tests have been updated to reflect this change.  It was necessary to
remove a unit test, since in the previous prompt "all - yes to all" was
actually treated as two options, split on the "-", i.e. both "all" and
"all - yes to all" were accepted answers to the prompt.  This is no
longer the case when the option is "All scripts", so only one unit test
is now required.
gep13 pushed a commit to lemonlambda/choco that referenced this issue May 29, 2025
After some discussion with the Chocolatey team, it has been decided that
the prompt option for "All - yes to all" is rather verbose, and the need
to include "yes to all" is unneeded, and not in line with how other
package managers do it when prompted with similar choices.

This commit removes the "All - yes to all" and replaces it with simply
"All scripts", which is more succinct.  In addition, the unit/pester
tests have been updated to reflect this change.  It was necessary to
remove a unit test, since in the previous prompt "all - yes to all" was
actually treated as two options, split on the "-", i.e. both "all" and
"all - yes to all" were accepted answers to the prompt.  This is no
longer the case when the option is "All scripts", so only one unit test
is now required.
@gep13 gep13 changed the title Change [A]ll in prompt to run script Replace verbose option for "[A]ll - yes to all" when installing packages May 29, 2025
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 a pull request may close this issue.

2 participants