<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">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 class="">> 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><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><div><br></div><div>-- </div><div>Ian.</div></div></div></div>