https://bugzilla.wikimedia.org/show_bug.cgi?id=53485

--- Comment #10 from Quim Gil <[email protected]> ---
Author, committer and reviewer are clear terms. "Merger" is confusing. 

As far as I know it's not a term used to describe a person, even less
contributors whose patch has been merged. If anything, it sounds as
contributors with +2 permissions in Gerrit that merge code from others.



But we really need to clean this terminology:

http://korma.wmflabs.org/browser/scr.html mentions "top openers" (what is
this?) and "top mergers". 

Clicking a contributor e.g.

http://korma.wmflabs.org/browser/people.html?id=278&name=Leslie%20Carr mentions

you see "commits" and "closed reviews", and none of the values match.

I would start by renaming "mergers" by "authors of merged code". We don't need
to force one word only if that word doesn't exist. It is more important to have
self-explanatory labels in metrics.

Alvaro, how do you think you are doing with this KPI? As of today none of the
questions of Comment #0 is answered yet. Do you think this will be ready by the
ECT Showcase meeting next week?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to