Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Import zsh-completion and polish a bit #395

Merged
merged 4 commits into from
Feb 8, 2024

Conversation

evelikov
Copy link
Collaborator

@evelikov evelikov commented Feb 8, 2024

This is the work of @hyperupcall over all #391 with a few fixes applied.

Since a few PRs landed, I thought about addressing the review comments myself, although since I cannot push to the initial (user?) repo here's a separate PR.

This way we can run it through the CI before merging.

hyperupcall and others added 4 commits February 8, 2024 21:11
Based on commit: a9386df5581487e5e34ecb3ce422bacf62d10c3d

[Emil Velikov]
 - rename the file and wire to install/uninstall

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
A bunch of the options got removed a few releases back - drop them.
While here apply minor polish - reorder couple of options and update few
comments

[Emil Velikov]
 - split/squash as applicable, commit message

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reorder some of the options to be somewhat consistent relative to the
dkms script and zsh completion.

[Emil Velikov]
 - split/squash as applicable, commit message

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
@evelikov evelikov merged commit bd8d53f into dell:master Feb 8, 2024
24 checks passed
@evelikov evelikov deleted the zsh-completion branch February 8, 2024 21:28
@evelikov evelikov mentioned this pull request Feb 8, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants