incubator-general mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Gruno <>
Subject Re: Github pull request hooks
Date Sat, 08 Feb 2014 11:51:12 GMT
Let me just get some information across to podlings wondering about this
new GitHub stuff we've added, and what is supported so far:

- It is now possible to relay new GitHub PRs, comments on PRs and
closing/reopening of PRs to your mailing list.

- It is also possible for all committers in a project to close PRs with
a commit message to the master/trunk branch of a project saying, for
instance, "this closes #1234 on GitHub" in the commit message.

- This GitHub feature is currently only supported for projects/podlings
that have their primary source repository in Git. If you have an SVN
mirror instead, we are not fully able to support GitHub PR relaying at
the time being, but will support it in the near future.

- You have to request PR comments to be enabled for your project,
preferably via a JIRA ticket (at ). By default, hooks have
been set up so that only PR opening/closing gets passed on. New setups
will include relaying of comments, but if your project has been set up
for a web hook in the past, you must request it be set up with comment
notification. This is a new default setting, and all projects that now
request PR notifications will automatically receive comments on PRs in
their respective mailing list as well. If you are unsure whether your
project is set up for relaying GitHub messages (and you want this), make
a new JIRA ticket to have it set up, rather one ticket too many than one
too few.

- Comments _should_ now be nested, so that each new comment has an email
reference to the original PR issue, meaning they will nest in mod_mbox
and your email client if you have nesting/threading enabled.

- We may, in the future, add the possibility to filter out certain
comments from being relayed to the ML (such as jenkins workflows etc),
but this will all depend on how this initial phase goes along.

With regards,

On 02/07/2014 10:02 PM, Daniel Gruno wrote:
> We had a bit of a snafu on adding PR titles to the subject lines, but
> it's been taken care of now.
> With regards,
> Daniel.
> On 02/07/2014 09:57 PM, Henry Saputra wrote:
>> Ah looks like the missing PR title only happen for the emails
>> describing comments from the Github pull requests.
>> - Henry
>> On Fri, Feb 7, 2014 at 12:55 PM, Henry Saputra <> wrote:
>>> Hi Jake,
>>> Looks like all the emails from the PR has the same subject:
>>> "[GitHub] incubator-spark pull request:"
>>> Is there a way to get the Title of the pull request included in the
>>> Subject line?
>>> Thanks,
>>> Henry
>>> On Fri, Feb 7, 2014 at 8:30 AM, Jake Farrell <> wrote:
>>>> I just wanted to follow up on my previous comments about the Github
>>>> webhooks not posting comments, Daniel Gruno and I have been debugging the
>>>> Github webhooks we had in place to send pull request notifications and we
>>>> have fixed it so comments for issues and pull requests will now go to the
>>>> dev@ lists. If anyone replies to this mail it will stay on the dev@ list.
>>>> -Jake
>>>> NOTE: This still remains a project/PMC responsibility to ensure
>>>> communication is recorded as this is coming from a 3rd party we do not
>>>> control. Communications should be encouraged to occur on our mailing lists.
>>>> We (infra) are working to make things easier and allow for better
>>>> integrations and are always open to anyone wanting to help contribute to
>>>> these efforts.
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail:
>> For additional commands, e-mail:
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message