<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 17, 2014 at 6:26 PM, <a href="mailto:marios@redhat.com" target="_blank">marios@redhat.com</a> <span dir="ltr"><<a href="mailto:mandreou@redhat.com" target="_blank">mandreou@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
as part of general housekeeping on our reviews, it was discussed at last<br>
week's meeting [1] that we should set workflow -1 for stale reviews<br>
(like gerrit used to do when I were a lad).<br>
<br>
The specific criteria discussed was 'items that have a -1 from a core<br>
but no response from author for 14 days'. This topic came up again<br>
during today's meeting and it wasn't clear if the intention was for<br>
cores to start enforcing this? So:<br>
<br>
Do we start setting WIP/workflow -1 for those reviews that have a -1<br>
from a core but no response from author for 14 days<br></blockquote><div><br></div><div>I'm in favour of doing this; as long as we make it clear that we're doing it to help us focus review effort on things that are under active development - it doesn't mean we think the patch shouldn't land, it just means we know it's not ready yet so we don't want reviewers to be looking at it until it moves forward.</div><div><br>For the sake of making sure new developers don't get put off, I'd like to see us leaving a comment explaining why we're WIPing the change and noting that uploading a new revision will remove the WIP automatically</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
thanks, marios<br>
<br>
[1]<br>
<a href="http://eavesdrop.openstack.org/meetings/tripleo/2014/tripleo.2014-09-09-19.04.log.html" target="_blank">http://eavesdrop.openstack.org/meetings/tripleo/2014/tripleo.2014-09-09-19.04.log.html</a><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">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>
</blockquote></div><br></div></div>