+1 Way to go!
On Wed, Nov 16, 2016 at 2:19 PM, Andrey Gura <ag...@apache.org> wrote: > +1 > > > > On Wed, Nov 16, 2016 at 1:44 PM, Pavel Tupitsyn <ptupit...@apache.org> > wrote: > > > Sergi, > > > > I don't think we should enforce it. > > "Official" means that it will be allowed, linked and described in wiki. > > > > It is up to contributor and reviewer to use it. > > For example, if the changes are small/trivial, creating a review in > > Upsource is not necessary. > > If the changes are complex and contributor has not created a review, > > reviewer may ask to do so. > > Etc etc. > > > > Thoughts? > > > > On Wed, Nov 16, 2016 at 1:31 PM, Sergi Vladykin < > sergi.vlady...@gmail.com> > > wrote: > > > > > Pavel, > > > > > > I'm not sure I understand the term "official code review tool", can you > > > please elaborate it? We will enforce everyone to register in Upsource > and > > > use it or what? > > > > > > Sergi > > > > > > 2016-11-16 13:22 GMT+03:00 Pavel Tupitsyn <ptupit...@apache.org>: > > > > > > > Proper link to the discussion: > > > > http://apache-ignite-developers.2346864.n4.nabble.com/ > > > > Code-Review-Tool-Proposal-Upsource-td12195.html > > > > > > > > On Wed, Nov 16, 2016 at 1:16 PM, Anton Vinogradov <a...@apache.org> > > wrote: > > > > > > > > > +1 > > > > > > > > > > On Wed, Nov 16, 2016 at 1:08 PM, Pavel Tupitsyn < > > ptupit...@apache.org> > > > > > wrote: > > > > > > > > > > > Following the discussion on Upsource [1], > > > > > > I would like to call a vote on accepting it as our official code > > > review > > > > > > tool. > > > > > > > > > > > > [ ] +1 approve > > > > > > [ ] +0 no opinion > > > > > > [ ] -1 disapprove (and reason why) > > > > > > > > > > > > This vote will go on for 5 days. > > > > > > > > > > > > [1] http://apache-ignite-developers.2346864.n4.nabble. > > > > > > com/Code-Review-Tool-Proposal-Upsource-td12195.html > > > > > > > > > > > > > > > > > > > > >
