On Thu, Aug 12, 2010 at 8:49 PM, Danese Cooper <dcooper(a)wikimedia.org> wrote:
I think you may be hallucinating. We are just now
hiring our second
code-reviewer, who is also thinking about ways to make the code review
process more transparent.
Is this someone with a lot of MediaWiki experience? If not, how are
they supposed to review code? I'd think it would be much more sound
to ask a couple of existing employees to pitch in instead, spending
part of their time reviewing and part of it coding. I can think of a
couple of people who are qualified enough. It doesn't make a lot of
sense to have someone who only reviews and doesn't code, unless
they've already done so much coding that they're very familiar with
the codebase and coding conventions anyway.
I also wonder how the review process could be more transparent, since
it's already completely public and open for anyone to join in (and
plenty of random non-developers do join in).