<div dir="ltr">All the points sound quite reasonable. I agree with Chris, the more reviewers read this, the better will be our review quality.<div><br></div><div>Do we have some kind of reviewing guide?, if we don't this could be an start.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">--</div><div class="gmail_extra">irc: ajo  / mangelajo<br clear="all"><div><div dir="ltr">Miguel Angel Ajo Pelayo<br>+34 636 52 25 69<br>skype: ajoajoajo</div></div>

<br><br><div class="gmail_quote">2013/11/6 Christopher Armstrong <span dir="ltr"><<a href="mailto:chris.armstrong@rackspace.com" target="_blank">chris.armstrong@rackspace.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="im">On Wed, Nov 6, 2013 at 2:34 AM, Radomir Dopieralski <span dir="ltr"><<a href="mailto:openstack@sheep.art.pl" target="_blank">openstack@sheep.art.pl</a>></span> wrote:<br></div><div class="gmail_extra">
<div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
I'm quite new in the OpenStack project, but I love it already. What is<br>
especially nifty is the automated review system -- I'm really impressed.<br>
I'm coming from a project in which we also did reviews of every change<br>
-- although it was mostly manual, and just one review was enough to<br>
merge -- and at some point in that project I noticed that it is very<br>
easy to give reviews that are unhelpful, frustrating and just get in the<br>
way of the actual work. I started paying attention to how I am reviewing<br>
code, and I managed to come up with several patterns that are bad. Once<br>
I know the patterns, it's easier to recognize when I'm doing something<br>
wrong and rethink the review. I would like to share the patterns that I<br>
noticed.<br>
<br></blockquote><div><br></div><div><br></div></div><div>Agreed on all points. I think Gerrit is nice in that it automates a lot of stuff, but unfortunately the workflow has not encouraged the best behavior for reviewers. This is a good list to follow -- but how can we be sure people will? This mailing list thread will only be seen by a small number of reviewers over the life of the project, I'm sure.</div>
<span class="HOEnZb"><font color="#888888">
<div> </div></font></span></div><span class="HOEnZb"><font color="#888888"><div><br></div>-- <br><div dir="ltr"><div>IRC: radix</div>Christopher Armstrong<div>Rackspace</div></div>
</font></span></div></div>
<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></blockquote></div><br></div></div>