Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (nike.apache.org: domain of braden@google.com designates
 209.85.214.49 as permitted sender)
MIME-Version: 1.0
Sender: braden@google.com
In-Reply-To: 
 <CABiQX1ULCdCdpQLgBXx56o=X1igRVbYfhVQ8S0B-2vgcJDypwg@mail.gmail.com>
References: 
 <CAOBL_k7N1r9fBJ3cFXdyV9XcKWbHbGjFmUQ+nMgHFafoTh-ANQ@mail.gmail.com>
	<CADVgkyNxdvoT9LCkwH6os3YznUtvCn4iBF_v4HZmyLwxSVayWQ@mail.gmail.com>
	<747428461720386118@unknownmsgid>
	<CABiQX1UUWk1N4T5gCayRoR8MkBO-Q797==7ktt5Znr-mdK=Kjw@mail.gmail.com>
	<CAOBL_k5VaOoHzvi+bV7Bm1yyWfj0ebtq_duAB=6NqKESG8Oxcw@mail.gmail.com>
	<50FDEEB8.4080003@apache.org>
	<CABiQX1ULCdCdpQLgBXx56o=X1igRVbYfhVQ8S0B-2vgcJDypwg@mail.gmail.com>
Date: Tue, 22 Jan 2013 13:19:38 -0500
Message-ID: 
 <CAMk5LLnvZsymmrRzcUb2-E0UxCqy5sGeE8k94JW441OwBkZAPg@mail.gmail.com>
Subject: Re: Can we talk about large features before they arrive in master?
From: Braden Shepherdson <braden@chromium.org>
To: dev@cordova.apache.org
Content-Type: multipart/alternative; boundary=001517478216eb5e6b04d3e49e22

--001517478216eb5e6b04d3e49e22
Content-Type: text/plain; charset=ISO-8859-1

Code reviews will generally sound good to Googlers, so long as we can keep
the turnaround down. It definitely keeps our code quality high on internal
projects, even if it is sometimes a pain to have to wait for a response and
do your own reviews. I've asked Michal and Andrew for over-the-shoulder iOS
reviews in the past, since I'm new to that platform.

I also want to apologize for the trouble with the ArrayBuffers on Android.
I was running into the bug with navigating in mobile-spec causing
deviceready not to fire, and had just changed my start page to the binary
echo test Michal wrote. It started working, so I cleaned up my debugging
and pushed. That was premature, since I broke some of the tests and hadn't
run the automatic tests. Gomen nasai.


Braden

On Tue, Jan 22, 2013 at 10:59 AM, Andrew Grieve <agrieve@chromium.org>wrote:

> ReviewBoard seems like a great fit to me! Let's try it out!
>
>
> On Mon, Jan 21, 2013 at 8:43 PM, Brian M Dube <bdube@apache.org> wrote:
>
> > On 01/21/2013 01:24 PM, Joe Bowser wrote:
> > > On Mon, Jan 21, 2013 at 1:06 PM, Andrew Grieve <agrieve@chromium.org>
> > wrote:
> > >> As for code reviews:
> > >>
> > >> I'd certainly be interested in more code-reviews. I think it's really
> > >> useful to get feedback on changes. The only time when it becomes a
> > burden
> > >> is when turn-around time gets too long (e.g. you submit for review and
> > no
> > >> one looks at it for over a day).
> > >>
> > >> Up until now, we've been using the github pull-request interface to
> have
> > >> others review our changes, but this isn't done very frequently. I also
> > >> don't love this approach because comments through it don't get posted
> > back
> > >> to the cordova mailing-list.
> > >
> > > I'm not super thrilled by this either, because our GitHub pull request
> > > system is completely broken since we can't actually close requests and
> > > indicate when we think things are a good idea or not. I think we
> > > should do what Android does with Gerrit (see
> > > https://android-review.googlesource.com) , but that'll involve
> > > additional infrastructure and another war with INFRA about whether
> > > it's the Apache way or whatever.
> >
> > An instance of ReviewBoard [1] exists at Apache [2], so I don't think it
> > means war about the Apache way. Is that something that could fill this
> > need?
> >
> > Brian
> >
> > [1] https://reviews.apache.org/
> > [2]
> > https://blogs.apache.org/infra/entry/reviewboard_instance_running_at_the
> >
>

--001517478216eb5e6b04d3e49e22--