Re: [PATCH] mergetool merge/skip/abort

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 21, 2009, 16:33 UTC
Message-ID: <alpine.DEB.1.00.0901211730530.3586@pacific.mpi-cbg.de>
In-Reply-To: <81bfc67a0901210817r49971c09kea9dc8ee5ae21572@mail.gmail.com>

Hi,

On Wed, 21 Jan 2009, Caleb Cushing wrote:

Show 9 quoted lines

> just a typo fix in the patch
> 
> >From 29c2873861a3aec8304529735307385e9e5c248a Mon Sep 17 00:00:00 2001
> From: Caleb Cushing <xenoterracide@gmail.com>
> Date: Tue, 20 Jan 2009 11:33:30 -0500
> Subject: [PATCH] mergetool merge/skip/abort
>  add functionality to skip merging a file or abort from the merge
> 
> ---

If you look at other patch submissions, you will find that they all do it differently. You will never see a "Date:" or "Subject:" line. You will see a much shorter commit subject. And a more verbose commit message.

Sometimes, you will see that a commit message is not quite informative, or does not explain the _motivation_ why it should be a good idea to do what the patch does, and me saying so very directly (but never meaning to offend).

Hth, Dscho