Thread (11 messages) flat view 11 messages, 2 authors, 2016-06-15

Re: [PATCH 1/4] Move redo merge code in a function

From: Johannes Schindelin <hidden>
Date: 2016-06-15 22:44:25

Hi,

On Sun, 23 Mar 2008, Jörg Sommer wrote:
Signed-off-by: Jörg Sommer <redacted>
I like the patch, but the commit message is not really meaningful if you 
do not read the patch.  Maybe you want to prefix it with "rebase -i:"?  
And _just_ maybe you want to give an explanation what you want to do with 
it?

(For reference, I think the recent patch series by Linus is a _wonderful_ 
example how to do it: it has a very informative cover letter, and each 
patch tells you more about what and why, leaving the how mostly to the 
patch.  Basically, it is a pleasure to read (and understand):

http://article.gmane.org/gmane.comp.version-control.git/77827)

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help