Re: [PATCH v2] branch: implement shortcut to delete last branch

2018-03-26 Thread Junio C Hamano
g...@matthieu-moy.fr writes: >> That said, I'd still be OK with it. > > I don't have objection either. FWIW, I do not even buy the "destructive commands should force spelling things out even more" argument in the first place. $ git checkout somelongtopicname $ work work work $ git

Re: [PATCH v2] branch: implement shortcut to delete last branch

2018-03-26 Thread git
Thanks for Cc-ing me, and sorry for not being very responsive these days :-\. Jeff King writes: > On Fri, Mar 23, 2018 at 10:40:34PM +, Aaron Greenberg wrote: > >> I can appreciate Matthieu's points on the use of "-" in destructive >> commands. As of this writing, git-merge supports the "-"

Re: [PATCH v2] branch: implement shortcut to delete last branch

2018-03-26 Thread Jeff King
On Fri, Mar 23, 2018 at 10:40:34PM +, Aaron Greenberg wrote: > I updated the commit message to include my first email's cover letter > and cleaned up the test. Thanks. This one looks good to me. > I can appreciate Matthieu's points on the use of "-" in destructive > commands. As of this

[PATCH v2] branch: implement shortcut to delete last branch

2018-03-23 Thread Aaron Greenberg
I updated the commit message to include my first email's cover letter and cleaned up the test. Copying Junio, since he also had good comments in the conversation you linked. I can appreciate Matthieu's points on the use of "-" in destructive commands. As of this writing, git-merge supports the