As Ashar pointed out this week, we've fallen behind in code review. On
Robla's page (
http://toolserver.org/~robla/crstats/crstats.html) you can
see that commits marked “new” is beginning to edge up again.
To help with code review, Roan introduced “sign-offs” for developers who
are not as familiar with the MediaWiki code base. I'm sure he'll
correct me if I'm wrong, but I would like to encourage any developer who
isn't ready to mark code “OK” to use the sign-off feature — to indicate
that they've tested or inspected the code.
If you're running trunk in your testing or (heaven forfend!)
production, please try to see if you're exercising new code and give us
feedback by marking the sign-off as “tested”. This is one of the best
ways to get acquainted with the code base — if you miss something in
your testing, we'll be sure to let you know!
Of course, if you've been reviewing code, THANK YOU and keep up the good
work.
Mark