-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Improve bash completion with compopt #5240
Improve bash completion with compopt #5240
Conversation
6118719
to
78b1ff0
Compare
Could you insert a commit at the beginning that updates We've added support for testing completions but it'd be a lot of work to go back and add them for all previous issues but I'd like for us to be making sure we cover them for future ones. |
Also, will this cause problems for people on older bash? See #5190. |
78b1ff0
to
f24a9cc
Compare
Moved the test change to 6e1fae9. I hope I fixed this correctly.
I think it would work on bash 4.0 or later. In my opinion, bash 4.0 is old enough to be a minimum supported version. |
The problem is macOS... |
clap_complete/tests/snapshots/home/static/exhaustive/bash/.bashrc
Outdated
Show resolved
Hide resolved
Sorry, I confused it with macPorts. Then, this change will cause the same compatibility issue on macOS. It is necessary to check bash version. |
de394a7
to
304ec9a
Compare
304ec9a
to
22bf919
Compare
I found out we cannot add tests for 22bf919 for now because white spaces are trimmed by runtime.complete(). |
22bf919
to
31e023f
Compare
Thank you very much! |
31e023f
to
13a7980
Compare
Thanks! |
This PR tries to improve bash completion support.
ValueHint::DirPath
ValueHint::Other