I see. Yeah, it would be interesting to know if any other project has considered formalizing this notion. It may also enable assignment of reviews (potentially automated using Josh's system) and maybe anonymity as well? On the downside, it isn't easily implemented and probably doesn't come without certain undesired side-effects.
Thanks, Nishkam On Wed, Aug 27, 2014 at 3:39 PM, Patrick Wendell <pwend...@gmail.com> wrote: > Hey Nishkam, > > To some extent we already have this process - many community members > help review patches and some earn a reputation where committer's will > take an LGTM from them seriously. I'd be interested in seeing if any > other projects recognize people who do this. > > - Patrick > > On Wed, Aug 27, 2014 at 2:36 PM, Nishkam Ravi <nr...@cloudera.com> wrote: > > Wonder if it would make sense to introduce a notion of 'Reviewers' as an > > intermediate tier to help distribute the load? While anyone can review > and > > comment on an open PR, reviewers would be able to say aye or nay subject > to > > confirmation by a committer? > > > > Thanks, > > Nishkam > > > > > > On Wed, Aug 27, 2014 at 2:11 PM, Nicholas Chammas > > <nicholas.cham...@gmail.com> wrote: > >> > >> On Tue, Aug 26, 2014 at 2:21 PM, Josh Rosen <rosenvi...@gmail.com> > wrote: > >> > >> > Last weekend, I started hacking on a Google App Engine app for helping > >> > with pull request review (screenshot: http://i.imgur.com/wwpZKYZ.png > ). > >> > > >> > >> BTW Josh, how can we stay up-to-date on your work on this tool? A JIRA > >> issue, perhaps? > >> > >> Nick > > > > >
