Job Snijders <job@xxxxxxxxxxxxx> writes: > On Tue, Aug 22, 2017 at 12:22:43PM -0700, Junio C Hamano wrote: >> Job Snijders <job@xxxxxxxxxxxxx> writes: >> > Add 'raw' blob_plain link in history overview >> > >> > Reviewed-by: Giuseppe Bilotta <giuseppe.bilotta@xxxxxxxxx> >> > Signed-off-by: Job Snijders <job@xxxxxxxxxxxxx> >> > >> > --- >> >> 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.
