William Morgan <wmorgan-git@xxxxxxxxxxxx> 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 <wmorgan-git@xxxxxxxxxxxx> > --- > 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 <wmorgan-git@xxxxxxxxxxxx> - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html