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

completions: added Solidigm plugin to autocomplete script #2178

Merged
merged 1 commit into from
Feb 2, 2024

Conversation

amashiana-sd
Copy link
Contributor

No description provided.

@igaw
Copy link
Collaborator

igaw commented Jan 19, 2024

Could you also update the zsh completion as well? Thanks!

@amashiana-sd
Copy link
Contributor Author

Could you also update the zsh completion as well? Thanks!

Am I correct in my understanding that zsh completion is the file called _nvme?

I see individual commands in there but nothing related to plugins. Do I just need to make sure all commands under Solidigm plugin are in that file?

@igaw
Copy link
Collaborator

igaw commented Jan 22, 2024

Yes, the _nvme file contains the tab completion for zsh. So far the only other plugin added supported is the ocp, though it doesn't look it is really implementing any option completion :( Well at least it support for the tab completion for the command.

In the long run I'd like to see zsh also implementing support for the all the plugins, that's why I ask for it :)

Thanks!

@amashiana-sd amashiana-sd force-pushed the solidigm_autocomplete branch 2 times, most recently from f3e6b69 to d5b32d8 Compare February 1, 2024 23:21
Updated both bash and zsh autocomplete scripts
Fixed copy-paste error under ocp section of zsh script

Signed-off-by: Anant Mashiana <[email protected]>
@igaw igaw merged commit c2fc029 into linux-nvme:master Feb 2, 2024
16 checks passed
@igaw
Copy link
Collaborator

igaw commented Feb 2, 2024

Very cool. Thanks!

@amashiana-sd amashiana-sd deleted the solidigm_autocomplete branch February 2, 2024 22:49
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