+1! On Thu, Mar 17, 2016 at 2:38 AM Khurram Faraaz <[email protected]> wrote:
> Thanks Till. :-) > > On Thu, Mar 17, 2016 at 12:04 PM, Mike Carey <[email protected]> wrote: > > > +1 :-) > > > > > > On 3/16/16 9:58 PM, Till Westmann wrote: > > > >> Hi, > >> > >> one of the most time consuming parts of code reviews seems to be the > >> repeated back-and-forth between the author and the reviewer(s). To try > to > >> reduce the need for repeated rounds I’d like to propose some DOs and > DON’Ts > >> for code reviews [1]. > >> These are the time-tested rules of an experienced academic advisor for > >> the reviews of papers and theses translated for code reviews :) > >> > >> Thoughts/comments? > >> > >> Cheers, > >> Till > >> > >> [1] https://cwiki.apache.org/confluence/display/ASTERIXDB/Code+Reviews > >> > > > > >
