On Sun, 2010-04-25 at 22:44 +0200, Patryk Zawadzki wrote:
> On Sun, Apr 25, 2010 at 7:59 PM, Curtis Hovey <sinzui...@verizon.net>
> wrote:
> > My suggestion is to support the Zeitgeist's community's culture of
> code
> > reviews. GNOME does not have an official code review tool. Neither
> does
> > GitHub, which is why projects that host in GitHub also use Launchpad
> for
> > code reviews.
> 
> Actually this is not true. GitHub lets you review any commit and the
> usual workflow is fork → commit → request pull → get review. 

Eh, github's pull requests are not really the same as a code review
system.  At least last time I looked at it.  You do a pull request and
the person you're requesting basically just gets a message that says,
"Hey dude, check out my awesome code!"  There isn't a nice UI for doing
the code review, with diffs that can be commented on and whatever.

Compare the link Lennart posted about gitorious, or compare the
Launchpad merge request system, to this:
  http://github.com/guides/pull-requests

/ Cody

_______________________________________________
desktop-devel-list mailing list
desktop-devel-list@gnome.org
http://mail.gnome.org/mailman/listinfo/desktop-devel-list

Reply via email to