From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Commit subject line |
Date: | 2013-05-03 18:54:23 |
Message-ID: | 518407DF.6060504@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 05/03/2013 02:43 PM, Tom Lane wrote:
> Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> writes:
>> On 03.05.2013 20:56, Bruce Momjian wrote:
>>> On Fri, May 3, 2013 at 01:42:33PM -0400, Andrew Dunstan wrote:
>>>> Yeah. The recommended style is to have the first line be 50 chars or
>>>> less, which is a bit unfortunate - it can be a challenge to keep to
>>>> that limit for a meaningful or comprehensive subject.
>> Oh, that's tight. I didn't know about the 50 char recommendation. I've
>> tried to keep mine < 76 chars, so that when you do "git log", it fits on
>> a 80 char display with the 4 char indentation that git log does.
> Yeah, that's news to me too. I've been using a 75-char line length for
> all my commit messages since we switched to git. It's frequently tough
> enough to get a useful headline into 75 chars --- I can't see trying to
> do 50.
man git-commit says:
Though not required, it’s a good idea to begin the commit message
with a single short (less than 50 character) line summarizing the
change, followed by a blank line and then a more thorough
description. Tools that turn commits into email, for example, use
the first line on the Subject: line and the rest of the commit in
the body.
I'd be happy to use 75 or whatever if we could convince the email tools
not to truncate the subject lines at 50.
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-05-03 19:07:11 | Re: Commit subject line |
Previous Message | Simon Riggs | 2013-05-03 18:52:08 | Re: corrupt pages detected by enabling checksums |