<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="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 class="im">On Nov 11, 2013, at 6:42 PM, Vishvananda Ishaya <<a href="mailto:vishvananda@gmail.com">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 class="HOEnZb"><font color="#888888"><br>
<br>
-- Ed Leafe<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">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>