<p dir="ltr">+1</p>
<div class="gmail_quote">On 15 Jan 2015 07:15, "Clint Byrum" <<a href="mailto:clint@fewbar.com">clint@fewbar.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello! It has been a while since we expanded our review team. The<br>
numbers aren't easy to read with recent dips caused by the summit and<br>
holidays. However, I believe James has demonstrated superb review skills<br>
and a commitment to the project that shows broad awareness of the<br>
project.<br>
<br>
Below are the results of a meta-review I did, selecting recent reviews<br>
by James with comments and a final score. I didn't find any reviews by<br>
James that I objected to.<br>
<br>
<a href="https://review.openstack.org/#/c/133554/" target="_blank">https://review.openstack.org/#/c/133554/</a> -- Took charge and provided<br>
valuable feedback. +2<br>
<a href="https://review.openstack.org/#/c/114360/" target="_blank">https://review.openstack.org/#/c/114360/</a> -- Good -1 asking for better<br>
commit message and then timely follow-up +1 with positive comments for<br>
more improvement. +2<br>
<a href="https://review.openstack.org/#/c/138947/" target="_blank">https://review.openstack.org/#/c/138947/</a> -- Simpler review, +1'd on Dec.<br>
19 and no follow-up since. Allowing 2 weeks for holiday vacation, this<br>
is only really about 7 - 10 working days and acceptable. +2<br>
<a href="https://review.openstack.org/#/c/146731/" target="_blank">https://review.openstack.org/#/c/146731/</a> -- Very thoughtful -1 review of<br>
recent change with alternatives to the approach submitted as patches.<br>
<a href="https://review.openstack.org/#/c/139876/" target="_blank">https://review.openstack.org/#/c/139876/</a> -- Simpler review, +1'd in<br>
agreement with everyone else. +1<br>
<a href="https://review.openstack.org/#/c/142621/" target="_blank">https://review.openstack.org/#/c/142621/</a> -- Thoughtful +1 with<br>
consideration for other reviewers. +2<br>
<a href="https://review.openstack.org/#/c/113983/" target="_blank">https://review.openstack.org/#/c/113983/</a> -- Thorough spec review with<br>
grammar pedantry noted as something that would not prevent a positive<br>
review score. +2<br>
<br>
All current tripleo-core members are invited to vote at this time. Thank<br>
you!<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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>
</blockquote></div>