-
Notifications
You must be signed in to change notification settings - Fork 749
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 bash auto completion support #370
Conversation
Closes #37 |
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.
Kompose should create the file if it's not exist.
./kompose completion
Error: open /etc/bash_completion.d/kompose.sh: no such file or directory
@ngtuna It should. Doesn't look like you're running as sudo / have user permission to /etc/bash_completion.d/ Try |
Ah see it now... It doesn't work on Mac. Could we add this support to darwin ? |
works on fedora for me 👍 |
Can we do it like I don't think that it is no nice write to I really like that stdout option, because if you put |
And one more. Some people are using ZSH (like me for example 😉 ) |
0815dcc
to
0107cc3
Compare
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.
👍
Tested ZSH and Bash on Linux, and just ZSH on MacOS
182e247
to
cd40a1c
Compare
Tests should be all green now (validate error with some unreachable code). I've also updated the README with the appropriate instructions on how to use this! Merge when you're ready 👍 |
Adds auto completion support that's generated via: ``` $ kompose complete ``` This file is added to /etc/bash_completion.d/kompose.sh
cd40a1c
to
a58cd69
Compare
LGTM @ngtuna can you confirm? |
Yes. Confirmed |
Adds auto completion support that's generated via:
This file is added to /etc/bash_completion.d/kompose.sh