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

Fix Zsh completion #583

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Fix Zsh completion #583

merged 1 commit into from
Aug 28, 2023

Conversation

marlonrichert
Copy link
Contributor

Currently, trying to get completion for brew services results in a completion function is not defined for command or alias: services and no actual completions. This is because #compdef brew in _brew_services conflicts with an identical statement in _brew, causing Zsh's compinit to not load _brew_services at all.

This commit fixes that by instead declaring that this file defines completion for brew services only.

Currently, trying to get completion for `brew services` results in `a completion function is not defined for command or alias: services` and no actual completions. This is because `#compdef brew` in _brew_services conflicts with an identical statement in _brew, causing Zsh's compinit to not load _brew_services at all.

This commit fixes that by instead declaring that this file defines completion for `brew services` only.
@SMillerDev
Copy link
Member

@MikeMcQuaid are these generated or manual?

@MikeMcQuaid
Copy link
Member

@SMillerDev manual but would be good to get them generated eventually.

@SMillerDev
Copy link
Member

Thanks @marlonrichert !

@SMillerDev SMillerDev merged commit ddae9cf into Homebrew:master Aug 28, 2023
5 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants