From: Jeff King Date: Fri, 20 Nov 2015 11:22:23 GMT Subject: Re: [PATCH 0/7] Modernize t9300-fast-import Message-ID: <20151120112223.GD11198@sigill.intra.peff.net> In-Reply-To: On Thu, Nov 19, 2015 at 10:54:11PM -0500, Eric Sunshine wrote: > On Thu, Nov 19, 2015 at 2:09 PM, Johannes Sixt wrote: > > Some time ago, I had to dig into t9300-fast-import and found it quite > > unhelpful that it does not follow our modern best-practices. This series > > brings it up-to-date. I thought I submit it now while it is quiet in > > the area. > > > > The larger patches are best viewed using -w -color-words because the > > regular patch text is ... overwhelming. > > For what it's worth, I read through the series (with care, I hope) and > didn't find anything amiss. Thanks, both. I gave it a quick read and didn't see anything wrong. The end result is certainly nicer to read. :) -Peff