Code reviews

[alex]

Hey there,

yesterday, at the last hangout, there was a little discussion about
how nice it is to have your code reviewed by someone else (not to
mention Guido's talk back in 2006
http://www.youtube.com/watch?v=sMql3Di4Kgc).

As a result, App Engine code reviews mailing list is the initial effort.

The idea is simple: push your code change lists to
codereview.appspot.com with
appengine-code-reviews+<lang>@googlegroups.com in CC and get it
reviewed by other people subscribed to the list, willing to do code
reviews or those that would like to exchange the favor of their code
being reviewed :)

A longer description can be found here:
https://groups.google.com/d/topic/appengine-code-reviews/w2z1KaD27Ao/discussion


What do you guys think?

-- alex

[Jim Morrison]

So for appengine sdk patches, why should I expect the
code reviews list to be better than filing bugs? Was anyone
from Google planning to pick up reviews on the patches?

--
Jim

[alex]

Jim, I immagine this more about projects/frameworks/libs/apps running on App Engine (hence the first part of the list name, "appengine-"), not for the SDK itself. This thing wasn't meant to substitute SDK bugs tracker.