Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (athena.apache.org: domain of shazron@gmail.com designates
 209.85.192.50 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAE6O_-ZvVNc2MnGHBKwQp0cg-i-8VMd3JbBM3yJXGxvGhjwXWA@mail.gmail.com>
References: 
 <CAE6O_-YDQqhm6AbsH=5Zpk4g2ZdSoeHOVi1ci40+6qiE5CUosg@mail.gmail.com>
 <CAK_TSX+NBY-49oYU=cpz67m1Kx3BP0viro0iS2C69OR7dprsBQ@mail.gmail.com>
 <CAEeF2Tc26NViJjf7exTb1DxjZYNfya0gP1t8hg7DUVvv68m9wg@mail.gmail.com>
 <CAE6O_-YowPno2v5_7=emDiUR_t=uuCQ7Ey7hEUynyqeMwnFY2w@mail.gmail.com>
 <CAK_TSXJpBAcqNHy5uLkkE11gwwdbm+bUVS6gAo9ZTyCDSGFatg@mail.gmail.com>
 <CAE6O_-ZvVNc2MnGHBKwQp0cg-i-8VMd3JbBM3yJXGxvGhjwXWA@mail.gmail.com>
From: Shazron <shazron@gmail.com>
Date: Mon, 3 Mar 2014 00:19:06 -0800
Message-ID: 
 <CAE6O_-ZFJTnBP+Q3BTQKNONOG6irL7=7k1kU8iovqLACOQxW0g@mail.gmail.com>
Subject: Re: Improved integration between Apache and GitHub
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a11c123866ba0ee04f3af7374

--001a11c123866ba0ee04f3af7374
Content-Type: text/plain; charset=ISO-8859-1

More clarification needed from :

> I assume this is for all the cordova repos?

I would assume yes.

>Do you just want a notification that something happens on github on IRC,
or do you want the actual comments relayed as well?

I would think just if something happens.

> Furthermore, please subscribe asfbot@urd.zones.apache.org to your dev@mailing list ( moderators can send an email to dev-subscribe-asfbot=
urd.zones.apache.org@cordova.apache.org and then ack the return mail)

Who are the mods on our list? Not sure.


On Tue, Feb 25, 2014 at 1:31 PM, Shazron <shazron@gmail.com> wrote:

> Full repo name not required, but implied. Just for completeness if someone
> is looking at the commit from Git ASF, not Github
>
>
> On Tuesday, February 25, 2014, Ian Clelland <iclelland@google.com> wrote:
>
>> On Tue, Feb 25, 2014 at 12:54 PM, Shazron <shazron@gmail.com> wrote:
>>
>> > Something I noticed from a commit from Ian:
>> > https://github.com/apache/cordova-plugin-file-transfer/pull/20
>> >
>> > The github user "asfgit" automatically closed the PR from that commit
>> -- I
>> > believe github scans the commit message to associate it with an issue/PR
>> > but not sure how it knows it was a merge. The relevant words in the
>> message
>> > are "closes #20".
>> >
>>
>> I was hoping that would happen when I wrote that commit line; I'll admit
>> that I didn't have very high hopes for it, though.
>>
>>
>> >
>> > Github has a way to refer to repos and issues, so this PR issue would be
>> > "apache/cordova-plugin-file-transfer#20" so if we added to the commit
>> > message "Closes apache/cordova-plugin-file-transfer#20" it should close
>> the
>> > corresponding PR/issue (I hope).
>> >
>>
>> Is the full repo name required? I thought that the example above shows
>> that
>> it isn't, and we just need to ensure that we use one of the magic verbs
>> and
>> mention the PR by number?
>>
>>
>> >
>> >
>> > On Tue, Feb 25, 2014 at 6:33 AM, Michal Mocny <mmocny@chromium.org>
>> wrote:
>> >
>> > > This is a sweet improvement, but if I read correct, doesn't address
>> our
>> > > biggest headaches:
>> > >
>> > > 1. You still cannot merge in PR from github
>> > > 2. We still cannot close PR ourselves
>> > > 3. Commits already automatically comment on corresponding JIRA
>> issues, so
>> > > this change merely bumps the timing of automatic JIRA references to
>> > > comments up a bit (helps discoverability, but doesn't in the end help
>> > > manage JIRA issues all that much).
>> > >
>> > > Perhaps the most interesting change is the better discoverability of
>> PR
>> > > from then dev ML without requiring explicit emails from the
>> contributor,
>> > > and our ability to reply directly from ML.
>> > >
>> > > -Michal
>> > >
>> > >
>> > > On Tue, Feb 25, 2014 at 7:24 AM, Ian Clelland <iclelland@chromium.org
>> > > >wrote:
>> > >
>> > > > Yes, and can we have a pony, too?
>> > > >
>> > > > Seriously, everything on that list is a huge improvement over the
>> > > > mostly-one-way-replication that we're working with now.
>> > > >
>> > > > Thanks for filing that.
>> > > >
>> > > >
>> > > > On Tue, Feb 25, 2014 at 12:17 AM, Shazron <shazron@gmail.com>
>> wrote:
>> > > >
>> > > > >
>> > > > >
>> > > >
>> > >
>> >
>> https://blogs.apache.org/infra/entry/improved_integration_between_apache_and
>> > > > >
>> > > > > Do we want everything on that list? I can file the issue...
>> > > > >
>> > > >
>> > >
>> >
>>
>

--001a11c123866ba0ee04f3af7374--