<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace">Thanks Matt for highlighting this (again). Please also see<br></div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace"><a href="http://openstack.markmail.org/thread/iaqsha7hbiitwqe2">http://openstack.markmail.org/thread/iaqsha7hbiitwqe2</a></div><div class="gmail_default" style="font-family:monospace,monospace"><a href="http://markmail.org/thread/k62gcehxg6gv5ep4">http://markmail.org/thread/k62gcehxg6gv5ep4</a></div><div class="gmail_default" style="font-family:monospace,monospace"><a href="http://markmail.org/thread/n753w3wljii67yug">http://markmail.org/thread/n753w3wljii67yug</a></div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace">When can we take some concrete action to stop these same kinds of things from coming up again and again?</div><div class="gmail_default" style="font-family:monospace,monospace"><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-family:courier new,monospace"></span><br style="font-family:courier new,monospace"><span style="font-family:courier new,monospace">-amrith</span><br><br></div></div></div></div>
<br><div class="gmail_quote">On Fri, Sep 22, 2017 at 8:10 AM, Tom Barron <span dir="ltr"><<a href="mailto:tpb@dyncloud.net" target="_blank">tpb@dyncloud.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
On 09/21/2017 10:21 PM, Matt Riedemann wrote:<br>
> I just wanted to highlight to people that there seems to be a series of<br>
> garbage patches in various projects [1] which are basically doing things<br>
> like fixing a single typo in a code comment, or very narrowly changing<br>
> http to https in links within docs.<br>
><br>
> Also +1ing ones own changes.<br>
><br>
> I've been trying to snuff these out in nova, but I see it's basically a<br>
> pattern widespread across several projects.<br>
><br>
> This is the boilerplate comment I give with my -1, feel free to employ<br>
> it yourself.<br>
><br>
> "Sorry but this isn't really a useful change. Fixing typos in code<br>
> comments when the context is still clear doesn't really help us, and<br>
> mostly seems like looking for padding stats on stackalytics. It's also a<br>
> drain on our CI environment.<br>
><br>
> If you fixed all of the typos in a single module, or in user-facing<br>
> documentation, or error messages, or something in the logs, or something<br>
> that actually doesn't make sense in code comments, then maybe, but this<br>
> isn't one of those things."<br>
><br>
> I'm not trying to be a jerk here, but this is annoying to the point I<br>
> felt the need to say something publicly.<br>
><br>
> [1] <a href="https://review.openstack.org/#/q/author:%255E.*inspur.*" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/q/author:%255E.*inspur.*</a><br>
><br>
<br>
</span>The boilerplate is helpful but have we considered putting something<br>
along these lines in official documentation so that reviewers can just<br>
point to it? It should then be clear to all that negative reviews on<br>
these grounds are not simply a function of the individual reviewer's<br>
judgment or personality.<br>
<br>
FWIW I think it is better not to attribute motivation in these cases.<br>
Perhaps the code submitter is trying to pad stats, but perhaps they are<br>
just a new contributor trying to learn the process with a "harmless"<br>
patch, or just a compulsive clean-upper who hasn't thought through the<br>
costs in reviewer time and CI resources.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<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>
</div></div></blockquote></div><br></div>