<div dir="ltr">Well said. This will encourage new developers.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 6, 2013 at 11:26 AM, Andrew Woodward <span dir="ltr"><<a href="mailto:xarses@gmail.com" target="_blank">xarses@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Great thread and very insightful. Yes; please make this more<br>
accessible to other reviewers. Adding this to the wiki is a great<br>
start.<br>
<br>
Andrew<br>
Mirantis<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Nov 6, 2013 at 11:05 AM, Pitucha, Stanislaw Izaak<br>
<<a href="mailto:stanislaw.pitucha@hp.com">stanislaw.pitucha@hp.com</a>> wrote:<br>
> How about putting it on a wiki and linking it from the top bar of gerrit itself? (call it "review guidelines" for example)<br>
> That way, even people who didn't look for it explicitly could find it.<br>
><br>
> Regards,<br>
> Stanisław Pitucha<br>
> Cloud Services<br>
> Hewlett Packard<br>
><br>
> -----Original Message-----<br>
> From: Sergey Skripnick [mailto:<a href="mailto:sskripnick@mirantis.com">sskripnick@mirantis.com</a>]<br>
> Sent: Wednesday, November 06, 2013 6:50 PM<br>
> To: OpenStack Development Mailing List (not for usage questions)<br>
> Subject: Re: [openstack-dev] Bad review patterns<br>
><br>
><br>
> This definitely should be somewhere in wiki or blog and in the bookmarks.<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>
><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>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
If google has done it, Google did it right!<br>
</font></span><div class="HOEnZb"><div class="h5"><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><br clear="all"><div><br></div>-- <br>Ravi<br>
</div>