-
Notifications
You must be signed in to change notification settings - Fork 20
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
Bash completion for flags #81
Conversation
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
I think the PR comments have been addressed. |
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Signed-off-by: Mabel Zhang <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-plugin1 #81 +/- ##
==============================================
Coverage ? 99.82%
==============================================
Files ? 15
Lines ? 584
Branches ? 0
==============================================
Hits ? 583
Misses ? 1
Partials ? 0 Continue to review full report at Codecov.
|
Signed-off-by: Louise Poubel <[email protected]>
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.
I pushed d69a07a to make sure we always install the CLI files, even if the ign
tool isn't found during build time.
More context in:
🎉 New feature
Part of gazebosim/gz-tools#1
Used together with gazebosim/gz-tools#87
Summary
Bash completion script for flags available to this subcommand.
This is a standalone function that depends on a new script in gazebosim/gz-tools#87 to call this.
Test it
After sourcing the dependent script in ign-tools, source the script in this PR:
If you just tab, you get the default system tab-completion that shows the files in the current directory:
If you type
-
, and then tab, you get the tab-completion for all the flags available for this subcommand:Then you can type partial flags, and it'll do tab-completion normally for you, such as
It also works for multiple flags, like
ign plugin --<flag1> --<flag2> ...
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.