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

Preserve existing user commands in zsh completion #680

Merged
merged 1 commit into from
Nov 15, 2017
Merged

Preserve existing user commands in zsh completion #680

merged 1 commit into from
Nov 15, 2017

Conversation

lhernanz
Copy link
Contributor

Hi,

this PR addresses #604. Instead of completely overwriting the user-commands variable, we retrieve its value first and we add the additional completion mechanisms later. This is needed when the user has several packages that try to use this completion extension mechanism. With the current code, the last loaded package overwrites the rest.

Regards

@spacewander
Copy link
Collaborator

@lhernanz
Could you cherry-pick your change and push -f, to avoid the trivial merge commit in the pull request?

@lhernanz
Copy link
Contributor Author

@spacewander
Done. You should only see a commit now. I should have created a topic branch but the change was so small...

@spacewander spacewander merged commit 6aecc2b into tj:master Nov 15, 2017
@spacewander
Copy link
Collaborator

@lhernanz
Merged. Thank you!

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