Re: Code review steps

2019-06-09 Thread Taewoo Kim
self-review > > is done? :-) > > > > Best, > > Taewoo > > > > > >> On Fri, Jun 7, 2019 at 12:51 AM Mike Carey wrote: > >> > >> +1 ! > >> > >>> On 6/6/19 6:07 PM, Till Westmann wrote: > >>> Hi,

Re: Code review steps

2019-06-09 Thread Till Westmann
't we add reviewers only if the self-review > is done? :-) > > Best, > Taewoo > > >> On Fri, Jun 7, 2019 at 12:51 AM Mike Carey wrote: >> >> +1 ! >> >>> On 6/6/19 6:07 PM, Till Westmann wrote: >>> Hi, >>> >>> I've p

Re: Code review steps

2019-06-07 Thread Taewoo Kim
That clearly works. Or why don't we add reviewers only if the self-review is done? :-) Best, Taewoo On Fri, Jun 7, 2019 at 12:51 AM Mike Carey wrote: > +1 ! > > On 6/6/19 6:07 PM, Till Westmann wrote: > > Hi, > > > > I've proposed an addition to our cod

Re: Code review steps

2019-06-07 Thread Mike Carey
+1 ! On 6/6/19 6:07 PM, Till Westmann wrote: Hi, I've proposed an addition to our code review steps on https://cwiki.apache.org/confluence/display/ASTERIXDB/Code+Reviews The only change from the steps we've been doing so far is the use of a "+1" code review by the author o

Code review steps

2019-06-06 Thread Till Westmann
Hi, I've proposed an addition to our code review steps on https://cwiki.apache.org/confluence/display/ASTERIXDB/Code+Reviews The only change from the steps we've been doing so far is the use of a "+1" code review by the author of a change to indicate that the self-review is done