Re: Template for commit messages

From: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
To: Andres Freund <andres(at)anarazel(dot)de>, Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Template for commit messages
Date: 2016-02-01 11:55:23
Message-ID: 56AF47AB.7040205@iki.fi
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 01/02/16 13:36, Andres Freund wrote:
> On 2016-01-28 06:03:02 -0500, Bruce Momjian wrote:
>> Reported-by:
>> Bug:
>> Author:
>> Reviewed-by:
>> Tested-by:
>> Backpatch-through:
>
> I personally, and I realize that I'm likely alone on that, would really
> like to see references to relevant threads. A year after a commit or
> such it often starts to get hard to know which threads a commit was
> about. Often it's easy enough if it's about a single feature, but
> bugfixes often originate in threads that have no directly corresponding
> thread. And often the commit happens a while after there's been activity
> in a thread. I spent days searching what thread "per discussion" in
> commit messages refers to.

+1. I also quite often look at a commit message, and then hunt the "per
discussion". A message-id would save some time there. I've seen you add
annotations like that to your commits, which is nice. I'll start doing
that myself. You've used the "Discussion: <message-id>" format for that,
so I'll go with that too. (Even if the message-id is given in free-form
in the paragraphs, that's helpful, but might as well use a fixed form.)

And that's a proposal that's not "we must all do it or it's useless". If
half of commit messages include that, then hunting the "per discussion"
for those commits becomes easier, and the rest are the same as now.

- Heikki

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2016-02-01 11:56:20 Re: Template for commit messages
Previous Message Michael Paquier 2016-02-01 11:53:56 Re: Template for commit messages