<div dir="ltr">Thanks for the patch Zane :)<div><br></div><div>-Kendall (diablo_rojo)</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 11, 2018 at 3:50 PM Zane Bitter <<a href="mailto:zbitter@redhat.com">zbitter@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 04/06/18 10:13, Zane Bitter wrote:<br>
> On 31/05/18 14:35, Julia Kreger wrote:<br>
>> Back to the topic of nitpicking!<br>
>><br>
>> I virtually sat down with Doug today and we hammered out the positive<br>
>> aspects that we feel like are the things that we as a community want<br>
>> to see as part of reviews coming out of this effort. The principles<br>
>> change[1] in governance has been updated as a result.<br>
>><br>
>> I think we are at a point where we have to state high level<br>
>> principles, and then also update guidelines or other context providing<br>
>> documentation to re-enforce some of items covered in this<br>
>> discussion... not just to educate new contributors, but to serve as a<br>
>> checkpoint for existing reviewers when making the decision as to how<br>
>> to vote change set. The question then becomes where would such<br>
>> guidelines or documentation best fit?<br>
> <br>
> I think the contributor guide is the logical place for it. Kendall <br>
> pointed out this existing section:<br>
> <br>
> <a href="https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html#reviewing-changes" rel="noreferrer" target="_blank">https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html#reviewing-changes</a> <br>
> <br>
> <br>
> It could go in there, or perhaps we separate out the parts about when to <br>
> use which review scores into a separate page from the mechanics of how <br>
> to use Gerrit.<br>
> <br>
>> Should we explicitly detail the<br>
>> cause/effect that occurs? Should we convey contributor perceptions, or<br>
>> maybe even just link to this thread as there has been a massive amount<br>
>> of feedback raising valid cases, points, and frustrations.<br>
>><br>
>> Personally, I'd lean towards a blended approach, but the question of<br>
>> where is one I'm unsure of. Thoughts?<br>
> <br>
> Let's crowdsource a set of heuristics that reviewers and contributors <br>
> should keep in mind when they're reviewing or having their changes <br>
> reviewed. I made a start on collecting ideas from this and past threads, <br>
> as well as my own reviewing experience, into a document that I've <br>
> presumptuously titled "How to Review Changes the OpenStack Way" (but <br>
> might be more accurately called "The Frank Sinatra Guide to Code Review" <br>
> at the moment):<br>
> <br>
> <a href="https://etherpad.openstack.org/p/review-the-openstack-way" rel="noreferrer" target="_blank">https://etherpad.openstack.org/p/review-the-openstack-way</a><br>
> <br>
> It's in an etherpad to make it easier for everyone to add their <br>
> suggestions and comments (folks in #openstack-tc have made some tweaks <br>
> already). After a suitable interval has passed to collect feedback, I'll <br>
> turn this into a contributor guide change.<br>
<br>
It's had a week to percolate (and I've seen quite a few people viewing <br>
the etherpad), so here is the review:<br>
<br>
<a href="https://review.openstack.org/574479" rel="noreferrer" target="_blank">https://review.openstack.org/574479</a><br>
<br>
- ZB<br>
<br>
> Have at it!<br>
> <br>
> cheers,<br>
> Zane.<br>
> <br>
>> -Julia<br>
>><br>
>> [1]: <a href="https://review.openstack.org/#/c/570940/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/570940/</a><br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>