systemml-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Luciano Resende <luckbr1...@gmail.com>
Subject Re: Pull Request Reviews
Date Mon, 13 Feb 2017 19:48:08 GMT
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 <dusenberrymw@gmail.com> 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 <deroneriksson@gmail.com>
> 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

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message