Re: [PATCH] Add 'raw' blob_plain link in history overview

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 22, 2017, 20:06 UTC
Message-ID: <xmqqvalf1i5v.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <20170822193520.GJ39363@Vurt.local>

Job Snijders <job@instituut.net> writes:

Show 14 quoted lines

> On Tue, Aug 22, 2017 at 12:22:43PM -0700, Junio C Hamano wrote:
>> Job Snijders <job@instituut.net> writes:
>> > Add 'raw' blob_plain link in history overview
>> >
>> > Reviewed-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
>> > Signed-off-by: Job Snijders <job@instituut.net>
>> >
>> > ---
>> 
>> Thanks; I somehow thought that your earlier one not just said what
>> it does (twice---that is not needed) but why this change is useful,
>> but that is lost in the patch description?
>
> We often work with very large plain text files...

You do not have to explain it to _me_ ;-) I wanted you to explain it to our history, instead of me manually tweaking your proposed log message in your patch with what you sent over e-mail in a follow up explanation like this one.

> ...
> immediately click to the 'raw' version, saving time and improving
> workflow.

Most of the above, or at least a condensed version of it, should be in the proposed log message. That is all I was saying.

Thanks.