<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">+1<br>
<br>
On 01/14/2015 07:34 PM, Robert Collins wrote:<br>
</div>
<blockquote
cite="mid:CAJ3HoZ0UMiHT5dJRvKhr4PXv6PSv15Ehyz7-CsumhgU0=zYwMQ@mail.gmail.com"
type="cite">
<p dir="ltr">+1</p>
<div class="gmail_quote">On 15 Jan 2015 07:15, "Clint Byrum" <<a
moz-do-not-send="true" 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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe"
target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a moz-do-not-send="true"
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>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: <a class="moz-txt-link-abbreviated" href="mailto:OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a>
<a class="moz-txt-link-freetext" href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a>
</pre>
</blockquote>
<br>
</body>
</html>