completions/zsh: Hide *instal from all commands #5851
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
brew style
with your changes locally?brew tests
with your changes locally?After my other completion PR was merged, someone contacted me to ask if i could help with this Discourse post. The user is bothered by the fact that the zsh completion function offers one-letter-short aliases like
instal
anduninstal
. I agree that they aren't helpful, as they produce confusing-looking output and force unnecessary disambiguation when the user enters (e.g.)unin
tab.These aliases were already hidden in the bash completion function in PR #3052. This change just does the equivalent for the zsh function.
Note for anyone testing this: If you have caching enabled, you may need to delete the cache files (under
~/.zcompcache
by default) to see the change right away.