From: Junio C Hamano Date: Wed, 09 Jan 2008 02:55:27 GMT Subject: Re: [PATCH] additional help when editing during interactive rebase Message-ID: <7vsl17pv1c.fsf@gitster.siamese.dyndns.org> In-Reply-To: <1199845915-sup-797@south> William Morgan writes: > I personally would have found this message useful the first time I used > git rebase --interactive. YMMV. Aside from this message being inappropriate as a proposed commit log message, I think what the patch tries to achieve is a worthy UI improvement. I would have removed those empty lines around the instruction if I were patching this, though. Losing 5 lines out of 25-line terminal was marginally Ok. Losing 9 lines 4 lines too many and is unacceptable. Thoughts? > Signed-off-by: William Morgan > --- > git-rebase--interactive.sh | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh > index acdcc54..d53d283 100755 > --- a/git-rebase--interactive.sh > +++ b/git-rebase--interactive.sh > @@ -263,6 +263,10 @@ do_next () { > warn > warn " git commit --amend" > warn > + warn "Once amended, continue with" > + warn > + warn " git rebase --continue" > + warn > exit 0 > ;; > squash|s) > -- > 1.5.4.rc2.68.ge708a-dirty > > > -- > William