Just a friendly reminder to everyone about their outstanding FIXMEs in Code Review (up to 97[1] from 63[2] beginning of June).
The following is a list of everyone who has a commit with a fixme on it:
tparscal - 11 awjrichards - 7 peter17 - 6 nimishg - 5 werdna - 4 catrope - 4 platonides - 4 demon - 4 huji - 4 kaldari - 4 siebrand - 2 jeroendedauw - 2 tstarling - 2 happy-melon - 2 sean_colombo - 2 papyromancer - 2 btongminh - 2 pdhanda - 2 neilk - 2 daniel - 2 maxsem - 2 tisane - 1 dale - 1 vyznev - 1 freakolowsky - 1 soxred93 - 1 nikerabbit - 1 mah - 1 brion - 1 aaron - 1 hartman - 1 diana - 1 dantman - 1 svip - 1 ialex - 1 leonsp - 1 philip - 1 vibber - 1 jdpond - 1 yaauie - 1 purodha - 1 rainman - 1 mgrabovsky - 1
TOTAL: 97
Please be sure to ping anyone you know isn't on this mailing list (like some contractors, or people working on special projects maybe?) so we can get everyone informed.
Cheers!
Siebrand
[1] http://www.mediawiki.org/w/index.php?limit=100&title=Special%3ACode%2FMe... [2] http://lists.wikimedia.org/pipermail/wikitech-l/2010-June/048066.html
Can we please exclude branches from this calculation?
- Trevor
On 8/20/10 3:54 AM, Siebrand Mazeland wrote:
Just a friendly reminder to everyone about their outstanding FIXMEs in Code Review (up to 97[1] from 63[2] beginning of June).
The following is a list of everyone who has a commit with a fixme on it:
tparscal - 11 awjrichards - 7 peter17 - 6 nimishg - 5 werdna - 4 catrope - 4 platonides - 4 demon - 4 huji - 4 kaldari - 4 siebrand - 2 jeroendedauw - 2 tstarling - 2 happy-melon - 2 sean_colombo - 2 papyromancer - 2 btongminh - 2 pdhanda - 2 neilk - 2 daniel - 2 maxsem - 2 tisane - 1 dale - 1 vyznev - 1 freakolowsky - 1 soxred93 - 1 nikerabbit - 1 mah - 1 brion - 1 aaron - 1 hartman - 1 diana - 1 dantman - 1 svip - 1 ialex - 1 leonsp - 1 philip - 1 vibber - 1 jdpond - 1 yaauie - 1 purodha - 1 rainman - 1 mgrabovsky - 1
TOTAL: 97
Please be sure to ping anyone you know isn't on this mailing list (like some contractors, or people working on special projects maybe?) so we can get everyone informed.
Cheers!
Siebrand
[1] http://www.mediawiki.org/w/index.php?limit=100&title=Special%3ACode%2FMe... [2] http://lists.wikimedia.org/pipermail/wikitech-l/2010-June/048066.html
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Trevor Parscal wrote:
Can we please exclude branches from this calculation?
I think this is what you want.
mysql> select cr_author, count(*) from code_rev where cr_status = 'fixme' and cr_path not like '/branches/%' group by cr_author order by count(*) desc; +--------------+----------+ | cr_author | count(*) | +--------------+----------+ | awjrichards | 7 | | kaldari | 5 | | nimishg | 5 | | werdna | 4 | | platonides | 4 | | huji | 4 | | catrope | 3 | | demon | 3 | | btongminh | 3 | | daniel | 2 | | pdhanda | 2 | | sean_colombo | 2 | | happy-melon | 2 | | tparscal | 2 | | jeroendedauw | 2 | | maxsem | 2 | | mgrabovsky | 1 | | rainman | 1 | | yaauie | 1 | | philip | 1 | | hartman | 1 | | ialex | 1 | | mah | 1 | | aaron | 1 | | soxred93 | 1 | | vyznev | 1 | | jdpond | 1 | | leonsp | 1 | | tisane | 1 | | thomasv | 1 | | brion | 1 | | nikerabbit | 1 | | siebrand | 1 | | tstarling | 1 | | diana | 1 | | dantman | 1 | | purodha | 1 | | svip | 1 | +--------------+----------+ 38 rows in set (0.07 sec)
The total is 74.
This chart could be made dynamic in a Toolserver tool, if there were interest. It could also possibly go into the extension, but the database would need some indices first.
MZMcBride
From: wikitech-l-bounces@lists.wikimedia.org On Behalf Of MZMcBride This chart could be made dynamic in a Toolserver tool
I nicked your query (hope you didn't mind :P) to make this: http://toolserver.org/~stwalkerster/codereview-fixme.php
Simon (stwalkerster)
On 8/20/10 10:56 PM, MZMcBride wrote:
Trevor Parscal wrote:
Can we please exclude branches from this calculation?
I think this is what you want.
mysql> select cr_author, count(*) from code_rev where cr_status = 'fixme' and cr_path not like '/branches/%' group by cr_author order by count(*) desc; +--------------+----------+ | cr_author | count(*) | +--------------+----------+ | awjrichards | 7 | | kaldari | 5 | | nimishg | 5 | | werdna | 4 | | platonides | 4 | | huji | 4 | | catrope | 3 | | demon | 3 | | btongminh | 3 | | daniel | 2 | | pdhanda | 2 | | sean_colombo | 2 | | happy-melon | 2 | | tparscal | 2 | | jeroendedauw | 2 | | maxsem | 2 | | mgrabovsky | 1 | | rainman | 1 | | yaauie | 1 | | philip | 1 | | hartman | 1 | | ialex | 1 | | mah | 1 | | aaron | 1 | | soxred93 | 1 | | vyznev | 1 | | jdpond | 1 | | leonsp | 1 | | tisane | 1 | | thomasv | 1 | | brion | 1 | | nikerabbit | 1 | | siebrand | 1 | | tstarling | 1 | | diana | 1 | | dantman | 1 | | purodha | 1 | | svip | 1 | +--------------+----------+ 38 rows in set (0.07 sec)
The total is 74.
This chart could be made dynamic in a Toolserver tool, if there were interest. It could also possibly go into the extension, but the database would need some indices first.
MZMcBride
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Yes, this is excellent, and much more useful than tracking all fixmes in the whole system. Generally we push people off into branches so they can develop in a less mission-critical environment. Tracking fixmes in those areas only servers to skew the numbers in a negative direction, or at least make it unclear how skewed they may or may not be.
Thanks for poking at this!
- Trevor
Siebrand Mazeland wrote:
Just a friendly reminder to everyone about their outstanding FIXMEs in
Code Review (up to 97[1] from 63[2] beginning of June).
The following is a list of everyone who has a commit with a fixme on it:
platonides - 4
r70434 is the $wgUser->getOption( 'stubthreshold' ) vs $wgUser->getStubThreshold(). It's not something I care about too much, nor does Aryeh care strongly... r70783 is fixed IMHO. r70809 is a documentation problem. Waiting for Nikerabbit answer... r70917 is a real bug (a bunch of parsertests failing) but I can't reproduce it, so need to grab Simetrical first to determine it.
wikitech-l@lists.wikimedia.org