git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] completion: suggest sequencer commands for revert

From
Junio C Hamano <gitster@pobox.com>
Date
May 29, 2015, 19:50 UTC
Message-ID
<xmqq1thzcgde.fsf@gitster.dls.corp.google.com>
In-Reply-To
<1432547975.4644.5.camel@virtuell-zuhause.de>
Thomas Braun <thomas.braun@virtuell-zuhause.de> writes:
Show 7 quoted lines
> Signed-off-by: Thomas Braun <thomas.braun@virtuell-zuhause.de>
> ---
>
> Hi,
>
> I added the sequencer commands for git revert. These are handy in case a git
> revert needs manual intervention.

This looks OK from a cursory read to me; asking opinions from those who have touched the file in the recent past (Ram also happens to be one of the people who were heavily involved in sequencer work).

Thanks.
Show 23 quoted lines
>
> Thanks,
> Thomas
>
>  contrib/completion/git-completion.bash | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/contrib/completion/git-completion.bash b/contrib/completion/git-completion.bash
> index bfc74e9..3c00acd 100644
> --- a/contrib/completion/git-completion.bash
> +++ b/contrib/completion/git-completion.bash
> @@ -2282,6 +2282,11 @@ _git_reset ()
>  
>  _git_revert ()
>  {
> +	local dir="$(__gitdir)"
> +	if [ -f "$dir"/REVERT_HEAD ]; then
> +		__gitcomp "--continue --quit --abort"
> +		return
> +	fi
>  	case "$cur" in
>  	--*)
>  		__gitcomp "--edit --mainline --no-edit --no-commit --signoff"
Previous: Thomas BraunNext: Ramkumar Ramachandra
Message 2 of 10 in “completion: suggest sequencer commands for revert”
  1. completion: suggest sequencer commands for revertThomas Braun, May 25, 2015
  2. Junio C HamanoMay 29, 2015
  3. Ramkumar RamachandraMay 29, 2015
  4. 0/2 completion: sequencer commandsThomas Braun, May 30, 2015
  5. 2/2 completion: suggest sequencer commands for revertThomas Braun, May 30, 2015
  6. 1/2 completion: Add sequencer functionThomas Braun, May 30, 2015
  7. SZEDER GáborMay 30, 2015
  8. Junio C HamanoJun 1, 2015
  9. SZEDER GáborJun 1, 2015
  10. 1/2 completion: Add sequencer functionThomas Braun, May 30, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.