Code review management

This is a WMF project page for our code review process. This takes a different form than a typical project page, as this is streamlining an ongoing process rather than a "project" with milestones and clear end date.

Main page: Code review

Justification
We have a large community of developers that are frustrated with the current velocity of our code reviews. On the flip side, we need to review changes made to the site carefully enough such that our operations team doesn't have to continuously recover from poor developer choices.

Plan
In the near term, we have recruited a larger team of existing and former staff (including Brion Vibber, Trevor Parscal, Roan Kattouw, and Mark as he comes back up to speed). In the longer term, we are brainstorming now how to best identify and train community members to take up some of the code review tasks and how to provide appropriate oversight to ensure that we continue to catch potential style, architecture, performance, and security flaws in submitted code.

Other than "just get cracking" (which we're doing), one other thing is to try to improve visibility into the problem. Rob Lanphier plans to start reporting on our current metrics based on some work that Bryan Tong Minh has already started. See code review status graph to see the number of "ok" commits plotted against the number of total commits in the "phase3" branch.