[PATCH 001 of 179 tests-refactor] run-tests: create classes for representing tests

Gregory Szorc gregory.szorc at gmail.com
Fri May 2 14:49:42 CDT 2014


On 5/2/2014 11:45 AM, Pierre-Yves David wrote:
>> [PATCH 001 of 179 tests-refactor] run-tests: create classes for
>> representing tests
> 
> Did you really patch bombed 179 patch in one go?
> 
> If so, that is super wrong, please keep the number of inflight patches
> low (patchbomb series of around 5 changesets).
> 

Well, only 85 right now because my smtp server dropped the connection
mid send and patchbomb didn't recover from that gracefully ;)

I was pretty diligent about making commits small. Most of these patches
require little cognitive ability to review. However, I wanted to have
the whole series available so reviewers could see the full direction
things are moving in.

Since you raised objections both here and in IRC, I'll hold off sending
the remaining patches until these are upstreamed.


More information about the Mercurial-devel mailing list