<div dir="ltr"><div class="gmail_extra">
<br><div class="gmail_quote">On Tue, May 29, 2018 at 4:26 PM, Ian Wells <span dir="ltr"><<a href="mailto:ijw.ubuntu@cack.org.uk" target="_blank">ijw.ubuntu@cack.org.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On 29 May 2018 at 14:53, Jeremy Stanley <span dir="ltr"><<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2018-05-29 15:25:01 -0500 (-0500), Jay S Bryant wrote:<br>
[...]<br>
<span>> Maybe it would be different now that I am a Core/PTL but in the past I had<br>
> been warned to be careful as it could be misinterpreted if I was changing<br>
> other people's patches or that it could look like I was trying to pad my<br>
> numbers. (I am a nit-picker though I do my best not to be.<br>
</span>[...]<br>
<br>
Most stats tracking goes by the Gerrit "Owner" metadata or the Git<br>
"Author" field, neither of which are modified in a typical new<br>
patchset workflow and so carry over from the original patchset #1<br>
(resetting Author requires creating a new commit from scratch or<br>
passing extra options to git to reset it, while changing the Owner<br>
needs a completely new Change-Id footer).<br></blockquote><div><br></div></span><div>We know this, but other people don't, so the comment is wise.  Also, arguably, if I badly fix someone else's patch, I'm making them look bad by leaving them with the 'credit' for my bad work, so it's important to be careful and tactful.  But the history is public record, at least.</div><span class="HOEnZb"><font color="#888888"><div><br></div></font></span></div></div></div></blockquote><div><br></div><div>If the patch is bad enough where I have to step in to rewrite, I'm making the submitter look bad no matter what. That makes everyone worse off.</div><div><br></div><div>Best,</div><div>Samuel<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="HOEnZb"><font color="#888888"><div></div><div>-- </div><div>Ian.</div></font></span></div></div></div>
<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>
<br></blockquote></div><br></div></div>