Thrift on Apache Reviewboard

[Diwaker Gupta <di...@maginatics.com>]

Folks,

I've submitted a request to add Thrift to Apache Reviewboard:
https://issues.apache.org/jira/browse/INFRA-3752

It will be great if some of the core Thrift devs (PMC?) could show
their support on that issue so we can do better code reviews for
Thrift.

cheers,
Diwaker

Re: Thrift on Apache Reviewboard

[Diwaker Gupta <di...@maginatics.com>]

Thanks Jake. Tony was quick to set us up!

Folks, I highly recommend using the Apache ReviewBoard to review
patches. If you haven't used ReviewBoard before, it's a great tool for
doing code reviews online. Once the code is ready for merging, you can
still attach the diff to Jira (for the ASF license).

I'm already playing around with it (2 minor reviews) -- it has been
configured to send out email to dev@ for new review requests.

Diwaker

On Wed, Jul 6, 2011 at 7:00 AM, Jake Farrell <jf...@apache.org> wrote:
> I commented on the ticket, thanks Diwaker for creating it.
>
>
>
> On Jul 5, 2011, at 6:32 PM, Diwaker Gupta wrote:
>
>> Folks,
>>
>> I've submitted a request to add Thrift to Apache Reviewboard:
>> https://issues.apache.org/jira/browse/INFRA-3752
>>
>> It will be great if some of the core Thrift devs (PMC?) could show
>> their support on that issue so we can do better code reviews for
>> Thrift.
>>
>> cheers,
>> Diwaker
>
>

Re: Thrift on Apache Reviewboard

[Jake Farrell <jf...@apache.org>]

I commented on the ticket, thanks Diwaker for creating it.



On Jul 5, 2011, at 6:32 PM, Diwaker Gupta wrote:

> Folks,
> 
> I've submitted a request to add Thrift to Apache Reviewboard:
> https://issues.apache.org/jira/browse/INFRA-3752
> 
> It will be great if some of the core Thrift devs (PMC?) could show
> their support on that issue so we can do better code reviews for
> Thrift.
> 
> cheers,
> Diwaker