Gil Quim (Nokia-MS/MtView) said the following on 23.08.2010 13:50: > On 08/23/2010 11:32 AM, Storm-Olsen Marius (Nokia-MS-Qt/Austin) wrote: >> As many of you already know, the developers of Qt often use their own >> in-house "pastebin" site called Codepaster, to do their code reviewing >> if not one-on-one. However, in the spirit of Open Governance we want >> to move this process into the public, and focus the usage on one solution. > > Just forwarding fast feedback received at > http://twitter.com/acruiz/status/21934588870 > > @quimgil Code review != pastebin. I would really encourage you to look > at ReviewBoard for proper code reviews :-) > > http://www.reviewboard.org/
While I agree that Reviewboard is a fine tool, I think it might be a bit too involved for the fast-pace code sharing and reviewing we tend to do. It would be a better fit with handling the Merge Requests for sure. But requiring personal accounts and signing in to do reviews, quick checking of patches and just general IRC commenting, I feel it's a tad too "involved", if you know what I mean. -- .marius _______________________________________________ Opengov mailing list Opengov@qt-labs.org http://lists.qt-labs.org/listinfo/opengov