Re: Pull Request Reviews

[Luciano Resende]

Apache usually have two ways for revieweing: REview then commit, Commit
then review.

If review is becoming a bottleneck, just use CTR,  maybe waiting for a PR
build to be green.


On Mon, Feb 13, 2017 at 10:03 AM  wrote:

> Thanks, Deron, for bringing up this topic!  PRs, and the associated
> discussions, are a critical part of any modern, successful open source
> project.  As Deron stated, anyone in the community should feel free to
> review PRs -- we want your thoughts and opinions and greatly appreciate
> your help!
>
> - Mike
>
> --
>
> Mike Dusenberry
> GitHub: github.com/dusenberrymw
> LinkedIn: linkedin.com/in/mikedusenberry
>
> Sent from my iPhone.
>
>
> > On Feb 3, 2017, at 6:55 PM, Deron Eriksson 
> wrote:
> >
> > Hi,
> >
> > Reviewing pull requests is a great way to contribute to the success of
> > SystemML. If you are involved in any way with SystemML, please consider
> > reviewing pull requests. Everyone can review pull requests, and it is a
> > great way to gain experience with the project.
> >
> > Thanks!
> > Deron
> >
> >
> > Username PRs Reviewed
> > mboehm7 134
> > dusenberrymw 112
> > deroneriksson 110
> > niketanpansare 40
> > gweidner 31
> > shirisht 26
> > akchinSTC 25
> > nakul02 23
> > bertholdreinwald 15
> > lresende 12
> > frreiss 12
> > fschueler 9
> > Wenpei 7
> > asurve 5
> > iyounus 4
> > MechCoder 3
> > MadisonJMyers 3
> > oza 2
> > fmakari 2
> > rightwaitforyou 2
> > ethanyxu 1
> > ckadner 1
> > petro-rudenko 1
> > hsaputra 1
> > FelixNeutatz 1
> > nishi-t 0
> > sandeep-n 0
> > romeokienzler 0
> > tgamal 0
> > taasawat 0
> > sourav-mazumder 0
> > kevin-bates 0
> > kakal 0
> > GrapeBaBa 0
> > objectadjective 0
> > nmanchev 0
> > jodersky 0
> > jdyer1 0
> > gmlewis 0
> > aloknsingh 0
> > akunft 0
> > ahmaurya 0
> >
> >
> > --
> > Deron Eriksson
> > Spark Technology Center
> > http://www.spark.tc/
>
-- 
Sent from my Mobile device

Re: Pull Request Reviews

[dusenberrymw]

Thanks, Deron, for bringing up this topic!  PRs, and the associated 
discussions, are a critical part of any modern, successful open source project. 
 As Deron stated, anyone in the community should feel free to review PRs -- we 
want your thoughts and opinions and greatly appreciate your help!

- Mike

--

Mike Dusenberry
GitHub: github.com/dusenberrymw
LinkedIn: linkedin.com/in/mikedusenberry

Sent from my iPhone.


> On Feb 3, 2017, at 6:55 PM, Deron Eriksson  wrote:
> 
> Hi,
> 
> Reviewing pull requests is a great way to contribute to the success of
> SystemML. If you are involved in any way with SystemML, please consider
> reviewing pull requests. Everyone can review pull requests, and it is a
> great way to gain experience with the project.
> 
> Thanks!
> Deron
> 
> 
> Username PRs Reviewed
> mboehm7 134
> dusenberrymw 112
> deroneriksson 110
> niketanpansare 40
> gweidner 31
> shirisht 26
> akchinSTC 25
> nakul02 23
> bertholdreinwald 15
> lresende 12
> frreiss 12
> fschueler 9
> Wenpei 7
> asurve 5
> iyounus 4
> MechCoder 3
> MadisonJMyers 3
> oza 2
> fmakari 2
> rightwaitforyou 2
> ethanyxu 1
> ckadner 1
> petro-rudenko 1
> hsaputra 1
> FelixNeutatz 1
> nishi-t 0
> sandeep-n 0
> romeokienzler 0
> tgamal 0
> taasawat 0
> sourav-mazumder 0
> kevin-bates 0
> kakal 0
> GrapeBaBa 0
> objectadjective 0
> nmanchev 0
> jodersky 0
> jdyer1 0
> gmlewis 0
> aloknsingh 0
> akunft 0
> ahmaurya 0
> 
> 
> -- 
> Deron Eriksson
> Spark Technology Center
> http://www.spark.tc/

Pull Request Reviews

[Deron Eriksson]

Hi,

Reviewing pull requests is a great way to contribute to the success of
SystemML. If you are involved in any way with SystemML, please consider
reviewing pull requests. Everyone can review pull requests, and it is a
great way to gain experience with the project.

Thanks!
Deron


Username PRs Reviewed
mboehm7 134
dusenberrymw 112
deroneriksson 110
niketanpansare 40
gweidner 31
shirisht 26
akchinSTC 25
nakul02 23
bertholdreinwald 15
lresende 12
frreiss 12
fschueler 9
Wenpei 7
asurve 5
iyounus 4
MechCoder 3
MadisonJMyers 3
oza 2
fmakari 2
rightwaitforyou 2
ethanyxu 1
ckadner 1
petro-rudenko 1
hsaputra 1
FelixNeutatz 1
nishi-t 0
sandeep-n 0
romeokienzler 0
tgamal 0
taasawat 0
sourav-mazumder 0
kevin-bates 0
kakal 0
GrapeBaBa 0
objectadjective 0
nmanchev 0
jodersky 0
jdyer1 0
gmlewis 0
aloknsingh 0
akunft 0
ahmaurya 0


-- 
Deron Eriksson
Spark Technology Center
http://www.spark.tc/