Re: [PATCH] fmt-merge-msg: show those involved in a merged series

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Mar 13, 2012, 05:23 UTC
Message-ID: <20120313052335.GA16664@burratino>
In-Reply-To: <7vipi9mfhx.fsf@alter.siamese.dyndns.org>

Junio C Hamano wrote:

> Having said that, I tweaked the automated rebuilding procedure a bit
> today, and made it annotate these merges with snippets from the
> branch description in the "What's cooking" document, so the commits
> on master..pu are hopefully "very pleasant"ly annotated.

Yes, that helps.

I also wonder if it would make sense to change

	Merge branch 'mm/push-default-switch-warning' into pu

	... description ...

	By Matthieu Moy
	* mm/push-default-switch-warning:
	  push: start warning upcoming default change for push.default

to

	Merge branch 'mm/push-default-switch-warning' into pu

	... description ...

	* 'mm/push-default-switch-warning' by Matthieu Moy:
	  push: start warning upcoming default change for push.default

which makes the author less distractingly prominent (since they will be right there in the log soon later once we hit the commits themselves) and makes the start of the list of commits easier to find by eye. This is what I was inarticulately hinting at in my message before. With two authors it still looks reasonable:

	* 'jc/pickaxe-ignore-case' by Junio C Hamano (2) and Ramsay Jones (1):
	  ctype.c: Fix a sparse warning
	  pickaxe: allow -i to search in patch case-insensitively
	  grep: use static trans-case table

If this seems worth a patch, I'd be glad to try it (and even gladder to learn that someone else already tried it).

Jonathan