<div dir="ltr">If I have a nit that doesn't affect things, I'll make a note of it and say if you do another patch I'd really like it fixed but also give the patch a vote. What I'll also do sometimes if I know the user or they are online I'll offer to fix things for them, that way they can see what I've done, I've sped things along and I haven't caused a simple change to take a long amount of time and reviews.<div><br></div><div>I think this is a great addition!</div><div><br></div><div>Thanks,<br><div><br></div><div>Amy (spotz)</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 29, 2018 at 6:55 AM, Julia Kreger <span dir="ltr"><<a href="mailto:juliaashleykreger@gmail.com" target="_blank">juliaashleykreger@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">During the Forum, the topic of review culture came up in session after<br>
session. During these discussions, the subject of our use of nitpicks<br>
were often raised as a point of contention and frustration, especially<br>
by community members that have left the community and that were<br>
attempting to re-engage the community. Contributors raised the point<br>
of review feedback requiring for extremely precise English, or<br>
compliance to a particular core reviewer's style preferences, which<br>
may not be the same as another core reviewer.<br>
<br>
These things are not just frustrating, but also very inhibiting for<br>
part time contributors such as students who may also be time limited.<br>
Or an operator who noticed something that was clearly a bug and that<br>
put forth a very minor fix and doesn't have the time to revise it over<br>
and over.<br>
<br>
While nitpicks do help guide and teach, the consensus seemed to be<br>
that we do need to shift the culture a little bit. As such, I've<br>
proposed a change to our principles[1] in governance that attempts to<br>
capture the essence and spirit of the nitpicking topic as a first<br>
step.<br>
<br>
-Julia<br>
---------<br>
[1]: <a href="https://review.openstack.org/570940" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>570940</a><br>
<br>
______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</blockquote></div><br></div>