Roland Hieber <rhi@xxxxxxxxxxxxxx> writes: > Even 'symbolic-ref' is only completed when > GIT_COMPLETION_SHOW_ALL_COMMANDS=1 is set, it currently defaults to > completing file names, which is not very helpful. Add a simple > completion function which completes options and refs. > > Signed-off-by: Roland Hieber <rhi@xxxxxxxxxxxxxx> > --- > PATCH v3: > - make use of __gitcomp_builtin instead of hard-coded options, and add > a test for it (thanks to Patrick Steinhardt) Nice. > - add empty line between test cases (thanks to Patrick Steinhardt) > > PATCH v2: https://lore.kernel.org/git/20240424215019.268208-1-rhi@xxxxxxxxxxxxxx > - no changes > > PATCH v1: https://lore.kernel.org/git/20240424210549.256256-1-rhi@xxxxxxxxxxxxxx/ > --- > contrib/completion/git-completion.bash | 11 +++++++++++ > t/t9902-completion.sh | 23 +++++++++++++++++++++++ > 2 files changed, 34 insertions(+) > > diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash > index 75193ded4bde..4d63fb6eeaf7 100644 > --- a/contrib/completion/git-completion.bash > +++ b/contrib/completion/git-completion.bash > @@ -3581,6 +3581,17 @@ _git_svn () > fi > } > > +_git_symbolic_ref () { > + case "$cur" in > + --*) > + __gitcomp_builtin symbolic-ref > + return > + ;; > + esac > + > + __git_complete_refs > +} > + > _git_tag () > { > local i c="$__git_cmd_idx" f=0 > diff --git a/t/t9902-completion.sh b/t/t9902-completion.sh > index 569cf2310434..963f865f27ed 100755 > --- a/t/t9902-completion.sh > +++ b/t/t9902-completion.sh > @@ -2518,6 +2518,29 @@ test_expect_success 'complete tree filename with metacharacters' ' > EOF > ' > > +test_expect_success 'symbolic-ref completes builtin options' ' > + test_completion "git symbolic-ref --d" <<-\EOF > + --delete Z > + EOF > +' > + > +test_expect_success 'symbolic-ref completes short ref names' ' > + test_completion "git symbolic-ref foo m" <<-\EOF > + main Z > + mybranch Z > + mytag Z > + EOF > +' > + > +test_expect_success 'symbolic-ref completes full ref names' ' > + test_completion "git symbolic-ref foo refs/" <<-\EOF > + refs/heads/main Z > + refs/heads/mybranch Z > + refs/tags/mytag Z > + refs/tags/A Z > + EOF > +' > + > test_expect_success PERL 'send-email' ' > test_completion "git send-email --cov" <<-\EOF && > --cover-from-description=Z
