User talk:AKlapper (WMF)/Code Review

Re: Recognize active CR+1 users based on korma statistics
Where are CR+1/CR-1 statistics in that page? The second table is about +2/-2 (seemingly excluding self-merges), the first table seems to be about any "Actions" as the header states, hence presumably it counts the number of patchsets submitted too, as well as any comment without CR tag. Nemo 12:34, 3 January 2016 (UTC)
 * Thanks for catching this. I've clarified the section that theese statistics do not exist yet. --AKlapper (WMF) (talk) 18:53, 4 January 2016 (UTC)

Re: Developers/Maintainers
Why do you call the page outdated? Nemo 12:38, 3 January 2016 (UTC)
 * The main issue I see with that page is that links to teams are practically useless: you can't add a team to a gerrit patch, only individuals. When there is only a team in the maintainer column, it usually means the component is unmaintained (that is, getting stuff reviewed is hard). Nemo 12:39, 3 January 2016 (UTC)
 * I call the page outdated because it is. I don't consider it fixable as it's broken by design. It lists deprecated Analytics projects, it lists undeployed stuff (like Nagios or Nimsoft), it lists stuff long-finished without any specific 'project' (like VirtualRestService), it lists people as maintainers who aren't active anymore (like Chris, Ryan) or moved on (like Chad, Timo), it's unclear to me how "Key extensions" were defined, or where those random items under "MediaWiki core" come from (what is "Extension support"? "Security"? What is "General/Unknown"?). And "Anyone can list themselves as in training [...], signaling that they want [...] achieve maintainership" might create wrong expectations as noone will follow up on it. --AKlapper (WMF) (talk) 22:32, 4 January 2016 (UTC)