From: Junio C Hamano Date: Tue, 22 Aug 2017 20:06:36 GMT Subject: Re: [PATCH] Add 'raw' blob_plain link in history overview Message-ID: In-Reply-To: <20170822193520.GJ39363@Vurt.local> Job Snijders writes: > On Tue, Aug 22, 2017 at 12:22:43PM -0700, Junio C Hamano wrote: >> Job Snijders writes: >> > Add 'raw' blob_plain link in history overview >> > >> > Reviewed-by: Giuseppe Bilotta >> > Signed-off-by: Job Snijders >> > >> > --- >> >> 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.