Re: [PATCH] mergetool merge/skip/abort

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Wed, 21 Jan 2009, Caleb Cushing wrote:

> just a typo fix in the patch
> 
> >From 29c2873861a3aec8304529735307385e9e5c248a Mon Sep 17 00:00:00 2001
> From: Caleb Cushing <xenoterracide@xxxxxxxxx>
> 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
--
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

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux