On Fri, Feb 10, 2012 at 5:08 AM, John Du Hart compwhizii@gmail.com wrote:
Unless you've been living under a rock (If you have, how's the wifi under there?) we're moving to git soon. Along with this will come a change in how we do code review. However, some people have expressed concerns over the usability of gerrit. Therefore I'd like to propose an alternative.
Phabricator is a code review tool written by and for Facebook that has been open sourced. For an introduction, see this: http://phabricator.com/docs/phabricator/article/Introduction.html
I've written up some documentation about Phabricator for our uses here: https://www.mediawiki.org/wiki/Phabricator
I would really like for some of our developers and reviewers to try this out as an alternative to gerrit. Personally I've found it much more pleasurable to work with than gerrit. If we think this might be a viable solution for us then I'd be willing to work on adding more integration (LDAP support and Unit testing integration).
Let me know if you have any questions or feedback. Thanks!
We have to consider how this will affect the operations team as well. We have a very hard requirement of review before merge, and we must have fine-grained permissions per-branch. It would be a serious shame to use phabricator for mediawiki development and gerrit for operations development. We need *more* integration between operations and development, not less.
We also need to consider Labs integration. I've been putting a lot of effort into building things around gerrit's model (which is fairly good, even if its UI is terrible). I'm fine with switching to something else as long as it's going to actually work.
- Ryan