-
-
Notifications
You must be signed in to change notification settings - Fork 467
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
Add nushell completion support #1030
Add nushell completion support #1030
Conversation
5224164
to
d6d93ee
Compare
d6d93ee
to
176db06
Compare
Previous committers on this portion of the code include @cnpryer and @mitsuhiko. Do you think you could take a look? (And I think @mitsuhiko is the only one with merge privileges?) |
I can review and merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I think you should disable generating |
@UnderCooled I think this should be reported as an issue to https://github.com/clap-rs/clap instead so that it simply doesn't generate such completions in the first place |
I reported this issue to upstream: clap-rs/clap#5592 |
This PR adds clap-based nushell completions.
Within the clap repository there is the
clap_complete_nushell
crate: https://github.com/clap-rs/clap/tree/master/clap_complete_nushell. I'm not sure why they have put it in a separate crate, and this means it requires some extra logic, but it's not too bad and as you can see I didn't need to change much.This should be backwards compatible.
I also update the docs and give an example.
Here what the output looks like (expand)