Re: [PATCH v2] completion: complete --delete, --move, and --remotes for git branch

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 9, 2016, 18:30 UTC
Message-ID: <xmqqh9ataj87.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <1470735284-5686-1-git-send-email-ville.skytta@iki.fi>

Ville Skyttä <ville.skytta@iki.fi> writes:

Show 20 quoted lines

> Signed-off-by: Ville Skyttä <ville.skytta@iki.fi>
> ---
>  contrib/completion/git-completion.bash | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
> index 6a187bc..76abbd1 100644
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -1008,8 +1008,8 @@ _git_branch ()
>  	while [ $c -lt $cword ]; do
>  		i="${words[c]}"
>  		case "$i" in
> -		-d|-m)	only_local_ref="y" ;;
> -		-r)	has_r="y" ;;
> +		-d|--delete|-m|--move)	only_local_ref="y" ;;
> +		-r|--remotes)		has_r="y" ;;
>  		esac
>  		((c++))
>  	done

Sounds sensible; we already had "-d" but not its fully-spelled variant, and you are adding it (together with its friends).

Will queue.

Show 9 quoted lines

> @@ -1023,7 +1023,7 @@ _git_branch ()
>  			--color --no-color --verbose --abbrev= --no-abbrev
>  			--track --no-track --contains --merged --no-merged
>  			--set-upstream-to= --edit-description --list
> -			--unset-upstream
> +			--unset-upstream --delete --move --remotes
>  			"
>  		;;
>  	*)