You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@beam.apache.org by Jean-Baptiste Onofré <jb...@nanthrax.net> on 2017/10/06 11:58:14 UTC

[DISCUSS] Switch to gitbox

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é
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com

Re: [DISCUSS] Switch to gitbox

Posted by tarush grover <ta...@gmail.com>.

+1

Regards,
Tarush

On Tue, 10 Oct 2017 at 1:19 PM, Jean-Baptiste Onofré <jb...@nanthrax.net>
wrote:

> Agree,
>
> However, I will a formal vote for the records. As soon as the vote passes,
> I
> will proceed with the next steps (with INFRA).
>
> Regards
> JB
>
> On 10/10/2017 12:31 AM, Kenneth Knowles wrote:
> > Sounds like wild agreement - JB will you be the one to proceed with the
> > process?
> >
> > On Mon, Oct 9, 2017 at 3:24 PM, Thomas Weise <th...@apache.org> wrote:
> >
> >> +1
> >>
> >> It will enable proper work with PRs in the github interface (like
> >> requesting reviewers, merging and closing inactive PRs, after github and
> >> ASF IDs are linked)
> >>
> >>
> >> On Mon, Oct 9, 2017 at 2:52 PM, Ismaël Mejía <ie...@gmail.com> wrote:
> >>
> >>> +1
> >>>
> >>> On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid>
> wrote:
> >>>
> >>>> +1.
> >>>>
> >>>> I do love myself a forcing function for passing tests.
> >>>>
> >>>> On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <aljoscha@apache.org
> >
> >>>> wrote:
> >>>>
> >>>>> +1
> >>>>>
> >>>>>> On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
> >>>>> 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
> >>>> <takidau@google.com.invalid
> >>>>>>
> >>>>>> wrote:
> >>>>>>
> >>>>>>> +1
> >>>>>>>
> >>>>>>> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax
> >> <relax@google.com.invalid
> >>>>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> +1
> >>>>>>>>
> >>>>>>>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
> >>>>> 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é <
> >>>> jb@nanthrax.net
> >>>>>>
> >>>>>>>>> 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é
> >>>>>>>>>> jbonofre@apache.org
> >>>>>>>>>> http://blog.nanthrax.net
> >>>>>>>>>> Talend - http://www.talend.com
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>
> >>>>>
> >>>>>
> >>>>
> >>>
> >>
> >
>
> --
> Jean-Baptiste Onofré
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>

Re: [DISCUSS] Switch to gitbox

Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.

Agree,

However, I will a formal vote for the records. As soon as the vote passes, I 
will proceed with the next steps (with INFRA).

Regards
JB

On 10/10/2017 12:31 AM, Kenneth Knowles wrote:
> Sounds like wild agreement - JB will you be the one to proceed with the
> process?
> 
> On Mon, Oct 9, 2017 at 3:24 PM, Thomas Weise <th...@apache.org> wrote:
> 
>> +1
>>
>> It will enable proper work with PRs in the github interface (like
>> requesting reviewers, merging and closing inactive PRs, after github and
>> ASF IDs are linked)
>>
>>
>> On Mon, Oct 9, 2017 at 2:52 PM, Ismaël Mejía <ie...@gmail.com> wrote:
>>
>>> +1
>>>
>>> On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid> wrote:
>>>
>>>> +1.
>>>>
>>>> I do love myself a forcing function for passing tests.
>>>>
>>>> On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <al...@apache.org>
>>>> wrote:
>>>>
>>>>> +1
>>>>>
>>>>>> On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
>>>>> 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
>>>> <takidau@google.com.invalid
>>>>>>
>>>>>> wrote:
>>>>>>
>>>>>>> +1
>>>>>>>
>>>>>>> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax
>> <relax@google.com.invalid
>>>>
>>>>>>> wrote:
>>>>>>>
>>>>>>>> +1
>>>>>>>>
>>>>>>>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
>>>>> 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é <
>>>> jb@nanthrax.net
>>>>>>
>>>>>>>>> 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é
>>>>>>>>>> jbonofre@apache.org
>>>>>>>>>> http://blog.nanthrax.net
>>>>>>>>>> Talend - http://www.talend.com
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>
>>>>>
>>>>
>>>
>>
> 

-- 
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com

Re: [DISCUSS] Switch to gitbox

Posted by Pei HE <pe...@apache.org>.

+1

On Tue, Oct 10, 2017 at 6:31 AM, Kenneth Knowles <kl...@google.com.invalid>
wrote:

> Sounds like wild agreement - JB will you be the one to proceed with the
> process?
>
> On Mon, Oct 9, 2017 at 3:24 PM, Thomas Weise <th...@apache.org> wrote:
>
> > +1
> >
> > It will enable proper work with PRs in the github interface (like
> > requesting reviewers, merging and closing inactive PRs, after github and
> > ASF IDs are linked)
> >
> >
> > On Mon, Oct 9, 2017 at 2:52 PM, Ismaël Mejía <ie...@gmail.com> wrote:
> >
> > > +1
> > >
> > > On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid>
> wrote:
> > >
> > > > +1.
> > > >
> > > > I do love myself a forcing function for passing tests.
> > > >
> > > > On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <
> aljoscha@apache.org>
> > > > wrote:
> > > >
> > > > > +1
> > > > >
> > > > > > On 6. Oct 2017, at 18:57, Kenneth Knowles <klk@google.com.INVALID
> >
> > > > > 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
> > > > <takidau@google.com.invalid
> > > > > >
> > > > > > wrote:
> > > > > >
> > > > > >> +1
> > > > > >>
> > > > > >> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax
> > <relax@google.com.invalid
> > > >
> > > > > >> wrote:
> > > > > >>
> > > > > >>> +1
> > > > > >>>
> > > > > >>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lcwik@google.com.invalid
> >
> > > > > 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é <
> > > > jb@nanthrax.net
> > > > > >
> > > > > >>>> 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é
> > > > > >>>>> jbonofre@apache.org
> > > > > >>>>> http://blog.nanthrax.net
> > > > > >>>>> Talend - http://www.talend.com
> > > > > >>>>>
> > > > > >>>>>
> > > > > >>>>
> > > > > >>>
> > > > > >>
> > > > >
> > > > >
> > > >
> > >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Kenneth Knowles <kl...@google.com.INVALID>.

Sounds like wild agreement - JB will you be the one to proceed with the
process?

On Mon, Oct 9, 2017 at 3:24 PM, Thomas Weise <th...@apache.org> wrote:

> +1
>
> It will enable proper work with PRs in the github interface (like
> requesting reviewers, merging and closing inactive PRs, after github and
> ASF IDs are linked)
>
>
> On Mon, Oct 9, 2017 at 2:52 PM, Ismaël Mejía <ie...@gmail.com> wrote:
>
> > +1
> >
> > On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid> wrote:
> >
> > > +1.
> > >
> > > I do love myself a forcing function for passing tests.
> > >
> > > On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <al...@apache.org>
> > > wrote:
> > >
> > > > +1
> > > >
> > > > > On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
> > > > 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
> > > <takidau@google.com.invalid
> > > > >
> > > > > wrote:
> > > > >
> > > > >> +1
> > > > >>
> > > > >> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax
> <relax@google.com.invalid
> > >
> > > > >> wrote:
> > > > >>
> > > > >>> +1
> > > > >>>
> > > > >>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
> > > > 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é <
> > > jb@nanthrax.net
> > > > >
> > > > >>>> 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é
> > > > >>>>> jbonofre@apache.org
> > > > >>>>> http://blog.nanthrax.net
> > > > >>>>> Talend - http://www.talend.com
> > > > >>>>>
> > > > >>>>>
> > > > >>>>
> > > > >>>
> > > > >>
> > > >
> > > >
> > >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Thomas Weise <th...@apache.org>.

+1

It will enable proper work with PRs in the github interface (like
requesting reviewers, merging and closing inactive PRs, after github and
ASF IDs are linked)


On Mon, Oct 9, 2017 at 2:52 PM, Ismaël Mejía <ie...@gmail.com> wrote:

> +1
>
> On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid> wrote:
>
> > +1.
> >
> > I do love myself a forcing function for passing tests.
> >
> > On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <al...@apache.org>
> > wrote:
> >
> > > +1
> > >
> > > > On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
> > > 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
> > <takidau@google.com.invalid
> > > >
> > > > wrote:
> > > >
> > > >> +1
> > > >>
> > > >> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <relax@google.com.invalid
> >
> > > >> wrote:
> > > >>
> > > >>> +1
> > > >>>
> > > >>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
> > > 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é <
> > jb@nanthrax.net
> > > >
> > > >>>> 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é
> > > >>>>> jbonofre@apache.org
> > > >>>>> http://blog.nanthrax.net
> > > >>>>> Talend - http://www.talend.com
> > > >>>>>
> > > >>>>>
> > > >>>>
> > > >>>
> > > >>
> > >
> > >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Ismaël Mejía <ie...@gmail.com>.

+1

On Oct 9, 2017 6:52 PM, "Thomas Groh" <tg...@google.com.invalid> wrote:

> +1.
>
> I do love myself a forcing function for passing tests.
>
> On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <al...@apache.org>
> wrote:
>
> > +1
> >
> > > On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
> > 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
> <takidau@google.com.invalid
> > >
> > > wrote:
> > >
> > >> +1
> > >>
> > >> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <re...@google.com.invalid>
> > >> wrote:
> > >>
> > >>> +1
> > >>>
> > >>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
> > 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é <
> jb@nanthrax.net
> > >
> > >>>> 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é
> > >>>>> jbonofre@apache.org
> > >>>>> http://blog.nanthrax.net
> > >>>>> Talend - http://www.talend.com
> > >>>>>
> > >>>>>
> > >>>>
> > >>>
> > >>
> >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Thomas Groh <tg...@google.com.INVALID>.

+1.

I do love myself a forcing function for passing tests.

On Mon, Oct 9, 2017 at 7:51 AM, Aljoscha Krettek <al...@apache.org>
wrote:

> +1
>
> > On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID>
> 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 <takidau@google.com.invalid
> >
> > wrote:
> >
> >> +1
> >>
> >> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <re...@google.com.invalid>
> >> wrote:
> >>
> >>> +1
> >>>
> >>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid>
> 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é <jb@nanthrax.net
> >
> >>>> 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é
> >>>>> jbonofre@apache.org
> >>>>> http://blog.nanthrax.net
> >>>>> Talend - http://www.talend.com
> >>>>>
> >>>>>
> >>>>
> >>>
> >>
>
>

Re: [DISCUSS] Switch to gitbox

Posted by Aljoscha Krettek <al...@apache.org>.

+1

> On 6. Oct 2017, at 18:57, Kenneth Knowles <kl...@google.com.INVALID> 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 <ta...@google.com.invalid>
> wrote:
> 
>> +1
>> 
>> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <re...@google.com.invalid>
>> wrote:
>> 
>>> +1
>>> 
>>> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid> 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é <jb...@nanthrax.net>
>>>> 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é
>>>>> jbonofre@apache.org
>>>>> http://blog.nanthrax.net
>>>>> Talend - http://www.talend.com
>>>>> 
>>>>> 
>>>> 
>>> 
>>

Re: [DISCUSS] Switch to gitbox

Posted by Kenneth Knowles <kl...@google.com.INVALID>.

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 <ta...@google.com.invalid>
wrote:

> +1
>
> On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <re...@google.com.invalid>
> wrote:
>
> > +1
> >
> > On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid> 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é <jb...@nanthrax.net>
> > > 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é
> > > > jbonofre@apache.org
> > > > http://blog.nanthrax.net
> > > > Talend - http://www.talend.com
> > > >
> > > >
> > >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Tyler Akidau <ta...@google.com.INVALID>.

+1

On Fri, Oct 6, 2017 at 8:54 AM Reuven Lax <re...@google.com.invalid> wrote:

> +1
>
> On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid> 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é <jb...@nanthrax.net>
> > 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é
> > > jbonofre@apache.org
> > > http://blog.nanthrax.net
> > > Talend - http://www.talend.com
> > >
> > >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Reuven Lax <re...@google.com.INVALID>.

+1

On Oct 6, 2017 4:51 PM, "Lukasz Cwik" <lc...@google.com.invalid> 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é <jb...@nanthrax.net>
> 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é
> > jbonofre@apache.org
> > http://blog.nanthrax.net
> > Talend - http://www.talend.com
> >
> >
>

Re: [DISCUSS] Switch to gitbox

Posted by Lukasz Cwik <lc...@google.com.INVALID>.

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é <jb...@nanthrax.net>
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é
> jbonofre@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>
>