<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 21, 2013 at 8:15 PM, Sean Roberts <span dir="ltr"><<a href="mailto:seanrob@yahoo-inc.com" target="_blank">seanrob@yahoo-inc.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I disagree with having to wait for another core member before +2 approve in all situations. For example, the training docs are very early, changing often, and with only a few active contributors. If I find a bug in the how to contribute instructions, I need to get it published right away.<br>


Once we have more members and have grown up a bit, I'm good with reviews requiring more eyes.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Yes Sean, I agree that the openstack-training folder is one exception here. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888">
~sean<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Aug 21, 2013, at 11:33, "Andreas Jaeger" <<a href="mailto:aj@suse.com">aj@suse.com</a>> wrote:<br>
<br>
> On 08/21/2013 05:25 PM, Anne Gentle wrote:<br>
>> Hi all,<br>
>> I'm updating the Documentation/HowTo wiki page with the following<br>
>> description of our review policy. Please let me know if there are any<br>
>> questions. I'll definitely update the wiki page as we refine our policy.<br>
>> ---<br>
>> All community members can review doc patches and give them +1 or -1.<br>
>> Documentation Core members can give +2 or -2 votes and also click<br>
>> Approve so that the doc goes live, published to <a href="http://docs.openstack.org" target="_blank">docs.openstack.org</a><br>
>> <<a href="http://docs.openstack.org" target="_blank">http://docs.openstack.org</a>> or <a href="http://api.openstack.org" target="_blank">api.openstack.org</a><br>
>> <<a href="http://api.openstack.org" target="_blank">http://api.openstack.org</a>>, based on the branch the patch is applied to.<br>
>><br>
>> Because the Docs team is small, core members have the choice when<br>
>> reviewing and must use best judgement before publishing. Generally<br>
>> speaking, core members will wait for one other core member to +2 a doc<br>
>> patch. However if the change is small and the build works, a doc core<br>
>> member can +2 and Approve a change without waiting for another reviewer.<br>
>> This is a judgement call so docs core people should exercise judgement<br>
>> when using this option.<br>
>><br>
>> Once two community members approve a doc patch, a doc core member can<br>
>> also review it and push it through without waiting for a second core member.<br>
>> ---<br>
><br>
> In other words - or as summary: Before a core member approves a patch,<br>
> it needs either another core members +2 or two +1s - and non-core<br>
> members are welcome to +1 a patch.<br>
><br>
> Andreas<br>
> --<br>
> Andreas Jaeger aj@{<a href="http://suse.com" target="_blank">suse.com</a>,<a href="http://opensuse.org" target="_blank">opensuse.org</a>} Twitter/Identica: jaegerandi<br>
>  SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany<br>
>   GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)<br>
>    GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126<br>
><br>
</div></div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> Openstack-docs mailing list<br>
> <a href="mailto:Openstack-docs@lists.openstack.org">Openstack-docs@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs</a><br>
</div></div></blockquote></div><br></div></div>