Sounds great. If I recall correctly, it means we could also us assignment / review requests to pass pull requests around, instead of "R: foo" comments.
On Fri, Oct 6, 2017 at 9:30 AM, Tyler Akidau <[email protected]> wrote: > +1 > > On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <[email protected]> > wrote: > > > +1 > > > > On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <[email protected]> wrote: > > > > > I think its a great idea and find that the mergebot works well on the > > > website. > > > Since gitbox enforces that the precommit checks pass, it would also be > a > > > good forcing function for the community to maintain reliably passing > > tests. > > > > > > On Fri, Oct 6, 2017 at 4:58 AM, Jean-Baptiste Onofré <[email protected]> > > > wrote: > > > > > > > Hi guys, > > > > > > > > We use Apache gitbox for the website and it works fine (as soon as > you > > > > linked your Apache and github with 2FA enabled). > > > > > > > > What do you think about moving to gitbox for the codebase itself ? > > > > > > > > It could speed up the review and merge for the PRs. > > > > > > > > Thoughts ? > > > > > > > > Regards > > > > JB > > > > -- > > > > Jean-Baptiste Onofré > > > > [email protected] > > > > http://blog.nanthrax.net > > > > Talend - http://www.talend.com > > > > > > > > > > > > > >
