You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Prasanna Santhanam <ts...@apache.org> on 2013/02/01 06:38:25 UTC
Re: ReviewBoard: Drive to clear pending reviews
On Fri, Feb 01, 2013 at 03:46:28AM +0530, Rohit Yadav wrote:
> Rant: Can reviewers who are committers commit patches sent for reviews
> when they can spare time to review and give "ship it" on the review?
> It becomes confusing whether the patch was committed or not.
>
Just a thought : reviewboard drives are best done weekly/fortnightly
rather than at the end of release. I'm not volunteering - just hoping
someone will pick that up. :)
--
Prasanna.,
Re: ReviewBoard: Drive to clear pending reviews
Posted by Animesh Chaturvedi <an...@citrix.com>.
Yes I will start doing it weekly
Thanks
Animesh
On Jan 31, 2013, at 9:39 PM, "Prasanna Santhanam" <ts...@apache.org> wrote:
> On Fri, Feb 01, 2013 at 03:46:28AM +0530, Rohit Yadav wrote:
>> Rant: Can reviewers who are committers commit patches sent for reviews
>> when they can spare time to review and give "ship it" on the review?
>> It becomes confusing whether the patch was committed or not.
>
> Just a thought : reviewboard drives are best done weekly/fortnightly
> rather than at the end of release. I'm not volunteering - just hoping
> someone will pick that up. :)
>
> --
> Prasanna.,