+1. I do love myself a forcing function for passing tests.
On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <[email protected]> wrote: > +1 > > > On 6. Oct 2017, at 18:57, Kenneth Knowles <[email protected]> > wrote: > > > > 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 > >>>>> > >>>>> > >>>> > >>> > >> > >
