Mailing-List: contact dev-help@karaf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@karaf.apache.org
Subject: Re: Automatic Jenkins builds for pull requests
To: dev@karaf.apache.org
References: <565C7B07.9080900@die-schneider.net>
 <565C8A5E.7010300@nanthrax.net> <565C93BA.9050002@die-schneider.net>
From: =?UTF-8?Q?Jean-Baptiste_Onofr=c3=a9?= <jb@nanthrax.net>
Message-ID: <565D3C32.4030104@nanthrax.net>
Date: Tue, 1 Dec 2015 07:20:34 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: <565C93BA.9050002@die-schneider.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit

At Spark, we have:
- a comment added by Jenkins with the result of the build including the PR
- a mail on the mailing list (on reviews) when the PR is ready to be 
reviewed (when the build is fine)

I will add this too.

Regards
JB

On 11/30/2015 07:21 PM, Christian Schneider wrote:
> I just set up the karaf-pr build. Lets see how it works when the first
> pull request is changed.
>
> It would be great to also have the Jira Integration. Not sure if this is
> what you meant but I think in jira the main thing missing is to see the
> commits. Of course seeing the pullrequests that match a jira will also
> make a lot of sense.
>
> Christian
>
> Am 30.11.2015 um 18:41 schrieb Jean-Baptiste Onofré:
>> On Spark, we use jenkins build for PR + plug to Jira and mailing.
>>
>> I will setup the same in Karaf.
>>
>> Regards
>> JB
>>
>> On 11/30/2015 05:36 PM, Christian Schneider wrote:
>>> Apache Artemis is doing automatic jenkins runs for all pull requests. I
>>> think this is really helpful as the contributor gets some good feedback
>>> about the state of his request.
>>> It also lowers the work of the committers as they can wait until the
>>> pull request is in a good state.
>>>
>>> If no one minds I will also install this for karaf.
>>>
>>> Christian
>>>
>>
>

-- 
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com