Mailing-List: contact dev-help@falcon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@falcon.apache.org
Subject: Re: [Announce] Github integration live
MIME-Version: 1.0
From: Balu Vellanki Bala <bvellanki@hortonworks.com>
To: "dev@falcon.apache.org" <dev@falcon.apache.org>
CC: "user@falcon.apache.org" <user@falcon.apache.org>
Thread-Topic: [Announce] Github integration live
Thread-Index: AQHRWld38kpayuTwsU+OYiJFJGA1b58SlFoAgAAvPICACHQLAIABL8oA
Date: Thu, 4 Feb 2016 20:53:10 +0000
Message-ID: <D2D8FA01.29AD2%bvellanki@hortonworks.com>
References: 
 <CAKCPnS4X0scAWB006GGgh7JcOaibM-oM1zt+xr4FJVHiB4kfqw@mail.gmail.com>
 <CAK5w241JpeFvm=1moX6_dtTxAVJrhk8HZma4ng3OjVh63i7m2Q@mail.gmail.com>
 <CAKCPnS7CC7EX8yq6hL-hRHrMq5eRf8Jj8XhXUi9xZza-hFj=-Q@mail.gmail.com>
 <CAKCPnS634+PN5RorM4HkSQ9JP_Bbtm7eh9NXafrMfZb_xCj1yQ@mail.gmail.com>
In-Reply-To: 
 <CAKCPnS634+PN5RorM4HkSQ9JP_Bbtm7eh9NXafrMfZb_xCj1yQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [184.23.216.210]
x-source-routing-agent: Processed
Content-Type: text/plain; charset="us-ascii"
Content-ID: <E0CC426D291AA84AA168A132AC01963F@exch080.serverpod.net>
Content-Transfer-Encoding: quoted-printable

 Ajay and Team,

Patches that are inflight will be committed old way, the new patches will
be reviewed and committed via pull request. Hope this is ok,

Thanks
Balu=20

On 2/3/16, 10:45 AM, "Ajay Yadav" <ajaynsit@gmail.com> wrote:

>Hello everyone,
>
>Some people are still uploading patches on reviewboard and JIRA. Request
>everyone to switch to pull request model. Let me know if you face any
>issues.
>
>On Fri, Jan 29, 2016 at 3:10 PM, Ajay Yadav <ajaynsit@gmail.com> wrote:
>
>> Looks like the email per review comment is not yet disabled, I am
>> following up with INFRA, sorry for the inconvenience.
>>
>> On Fri, Jan 29, 2016 at 12:21 PM, Pallavi Rao <pallavi.rao@inmobi.com>
>> wrote:
>>
>>> Thanks Ajay for enabling this! Simplifies the whole patch upload,
>>>review,
>>> commit process.
>>>
>>> Will start using right away :-).
>>>
>>> On Fri, Jan 29, 2016 at 11:09 AM, Ajay Yadava <ajayyadava@apache.org>
>>> wrote:
>>>
>>>> Hello everyone,
>>>>
>>>> Good news!! Github integration is complete and ready for use. As part
>>>>of
>>>> this some more good stuff got done.
>>>>
>>>>    1. A new tool to merge pull requests has been created. It *squashes
>>>>    commits*, *reformats the commit messages in standard format*, *adds
>>>>    reviewers, retains original contributor as author of the commit
>>>>*etc.
>>>>    Usage guide has been made available on wiki
>>>>   =20
>>>><https://cwiki.apache.org/confluence/display/FALCON/How+to+merge+pull+r
>>>>equests>
>>>>    .
>>>>    2. New merge tool also automates the closure of the relevant JIRA
>>>>    with fix version.
>>>>    3. A pre commit build has also been configured. No more testing
>>>>    patches on your local box.
>>>>    4. How to Contribute guide
>>>>   =20
>>>><https://cwiki.apache.org/confluence/display/FALCON/How+To+Contribute>
>>>>has
>>>>    been updated with several new sections for non code contributions
>>>>and
>>>>    github pull request model.
>>>>    5. README file has been updated. It now parses markdown
>>>>syntax(looks
>>>>    lot better in github now) and contains the link to how to
>>>>contribute guide,
>>>>    change log etc.
>>>>
>>>> *How it works?*
>>>> With the new integration in place, whenever a new pull request is
>>>> created, a pre commit build will be automatically triggered and
>>>>result will
>>>> be displayed along with the "conflict status"(see attachment).
>>>>
>>>> Also every time a pull request *which contains JIRA id* is created asf
>>>> bot puts these details in the corresponding JIRA. See FALCON-1770
>>>> <https://issues.apache.org/jira/browse/FALCON-1770> for an example.
>>>>
>>>> To avoid noise on the mailing list we have disabled the emails
>>>>whenever
>>>> a comment is made on the pull request. You should watch the
>>>>repository and
>>>> use github notifications to see all updates.
>>>>
>>>> *Key things to take care of for pull requests*:
>>>>
>>>>    - Mention JIRA id in your pull request see this pr
>>>>    <https://github.com/apache/falcon/pull/13>for reference
>>>>    - Don't forget to update the CHANGES.txt
>>>>    - You *don't need* to squash commit messages, but please make
>>>>    meaningful commit messages for each update.
>>>>
>>>>
>>>> I will recommend all contributors to start using the new pull request
>>>> method so that they are reflected as the author of their
>>>>contributions in
>>>> the github/git.
>>>>
>>>> *[image: Screen Shot 2016-01-23 at 11.45.02 PM.png]*
>>>>
>>>> Cheers
>>>> Ajay Yadava
>>>>
>>>
>>>
>>> _____________________________________________________________
>>> The information contained in this communication is intended solely for
>>> the use of the individual or entity to whom it is addressed and others
>>> authorized to receive it. It may contain confidential or legally
>>>privileged
>>> information. If you are not the intended recipient you are hereby
>>>notified
>>> that any disclosure, copying, distribution or taking any action in
>>>reliance
>>> on the contents of this information is strictly prohibited and may be
>>> unlawful. If you have received this communication in error, please
>>>notify
>>> us immediately by responding to this email and then delete it from your
>>> system. The firm is neither liable for the proper and complete
>>>transmission
>>> of the information contained in this communication nor for any delay
>>>in its
>>> receipt.
>>
>>
>>