<div dir="ltr"><br><div>To be clear, that was a +1 for Mark's suggestion:</div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">> In cases like that, I'd be of a mind to go "+2 Awesome! Thanks for</span><br style="font-family:arial,sans-serif;font-size:13px">

<span style="font-family:arial,sans-serif;font-size:13px">> catching this! It would be great to have a unit test for this, but it's</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">> clear the current code is broken so I'm fine with merging the fix</span><br style="font-family:arial,sans-serif;font-size:13px">

<span style="font-family:arial,sans-serif;font-size:13px">> without a test". You could say it's now the reviewers responsibility to</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">> merge a test, but if that requirement then turns off reviewers even</span><br style="font-family:arial,sans-serif;font-size:13px">

<span style="font-family:arial,sans-serif;font-size:13px">> reviewing such a patch, then that doesn't help either.</span><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 12 November 2013 11:29, Michael Bright <span dir="ltr"><<a href="mailto:mjbrightfr@gmail.com" target="_blank">mjbrightfr@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div>+1 also.</div><div>I spent less than half the time on my first fix (so far) understanding the problem, reproducing it, coding it and learning about the code review system.</div>

<div><br></div><div>Much more than half the time was spent on reverse engineering existing tests to be able to add new ones (which had to use features not used by the existing tests) and asking for advice even on where to add the tests.</div>


<div><br></div><div>It would have been more efficient for everyone had some test examples been proposed to me.</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">

On 12 November 2013 03:34, Ed Leafe <span dir="ltr"><<a href="mailto:ed@openstack.org" target="_blank">ed@openstack.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Nov 11, 2013, at 6:42 PM, Vishvananda Ishaya <<a href="mailto:vishvananda@gmail.com" target="_blank">vishvananda@gmail.com</a>> wrote:<br>



<br>
> It also gives the submitter a specific example of a well-written test, which<br>
> can be a faster way to learn than forcing them to get there via trial and error.<br>
<br>
</div>+1. Implementing a policy that has as the end effect more knowledgeable contributors is a big win.<br>
<span><font color="#888888"><br>
<br>
-- Ed Leafe<br>
</font></span><div><div><br>
<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>