On Tue, Oct 30, 2012 at 4:18 PM, Matt Mackall <span dir="ltr"><<a href="mailto:mpm@selenic.com" target="_blank">mpm@selenic.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":1ru">And yet I'm seeing new patches submitted on a nearly daily basis from subscribers to this list<br>
that ignore this, some of them being accepted by crew members who<br>
haven't read it either.<br></div></blockquote></div><br><div>Since this is clearly aimed at me:</div><div><br></div><div>I read that wiki page, and I don't push changes that add new tests. (Also, I am bummed at the implied claim that an oversight on one small point means that I must either have not read the document or have chosen to ignore it.)</div>
<div><br></div><div>The commit that you got riled up over does follow the "fold your test into an appropriate existing test" dictum from that wiki page. You're right that it doesn't adapt a pre-existing hunk of a test so that it tests several things at once. This is simply because that is a subtle point that is quite easy to overlook during a review, not because I haven't read that policy or disagree with its intent.</div>