You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@netbeans.apache.org by Ate Douma <at...@douma.nu> on 2016/10/23 14:48:41 UTC
Fwd: Submitting and Handling Github PRs
I'm forwarding a reference about handling PRs from Suneel Marthi (mentor)
for the Streams podling, which IMO might also be useful for the NetBeans
community to consider.
Regards, Ate
-------- Forwarded Message --------
Subject: Submitting and Handling Github PRs
Date: Sat, 22 Oct 2016 16:57:33 -0400
From: Suneel Marthi <sm...@apache.org>
Reply-To: dev@streams.incubator.apache.org
To: dev@streams.incubator.apache.org
Wanted to bring up 2 things about managing PRs:
1. Squash all commits before merging PRs to master
2. Include the jira in the final commit before merging to master
Most projects (TLP and Incubator) do it - Mahout, Flink, Pirk, PredictionIO.
For reference, see
https://mahout.apache.org/developers/github.html for how its being done on
Mahout.
Re: Submitting and Handling Github PRs
Posted by Sven Reimers <sv...@gmail.com>.
Awesome.
Thanks for sharing
Sven
Am 23.10.2016 18:53 schrieb "Wade Chandler" <co...@wadechandler.com>:
> Great, thanks Ate.
>
> Wade
>
>
> ===================
>
> Wade Chandler
> e: consult@wadechandler.com
>
>
>
> > On Oct 23, 2016, at 10:48, Ate Douma <at...@douma.nu> wrote:
> >
> > I'm forwarding a reference about handling PRs from Suneel Marthi (mentor)
> > for the Streams podling, which IMO might also be useful for the NetBeans
> > community to consider.
> >
> > Regards, Ate
> >
> > -------- Forwarded Message --------
> > Subject: Submitting and Handling Github PRs
> > Date: Sat, 22 Oct 2016 16:57:33 -0400
> > From: Suneel Marthi <sm...@apache.org>
> > Reply-To: dev@streams.incubator.apache.org
> > To: dev@streams.incubator.apache.org
> >
> > Wanted to bring up 2 things about managing PRs:
> >
> > 1. Squash all commits before merging PRs to master
> >
> > 2. Include the jira in the final commit before merging to master
> >
> > Most projects (TLP and Incubator) do it - Mahout, Flink, Pirk,
> PredictionIO.
> >
> > For reference, see
> >
> > https://mahout.apache.org/developers/github.html for how its being done
> on
> > Mahout.
> >
>
>
Re: Submitting and Handling Github PRs
Posted by Wade Chandler <co...@wadechandler.com>.
Great, thanks Ate.
Wade
===================
Wade Chandler
e: consult@wadechandler.com
> On Oct 23, 2016, at 10:48, Ate Douma <at...@douma.nu> wrote:
>
> I'm forwarding a reference about handling PRs from Suneel Marthi (mentor)
> for the Streams podling, which IMO might also be useful for the NetBeans
> community to consider.
>
> Regards, Ate
>
> -------- Forwarded Message --------
> Subject: Submitting and Handling Github PRs
> Date: Sat, 22 Oct 2016 16:57:33 -0400
> From: Suneel Marthi <sm...@apache.org>
> Reply-To: dev@streams.incubator.apache.org
> To: dev@streams.incubator.apache.org
>
> Wanted to bring up 2 things about managing PRs:
>
> 1. Squash all commits before merging PRs to master
>
> 2. Include the jira in the final commit before merging to master
>
> Most projects (TLP and Incubator) do it - Mahout, Flink, Pirk, PredictionIO.
>
> For reference, see
>
> https://mahout.apache.org/developers/github.html for how its being done on
> Mahout.
>