[Yanel-dev] Review then Commit [WAS: Re: Debug instead of Warning]

Michael Wechner michael.wechner at wyona.com
Mon Jan 31 20:34:55 CET 2011


Hi Cedric

I have started a review checklist at

http://127.0.0.1:8080/yanel/yanel-website/en/development/processes/rtc.html

Sending        
src/realms/yanel-website/content/db90075e-74d0-4d8c-bdff-661e93af77cb
Transmitting file data .
Committed revision 56409.

As mentioned by you privately I will have a look at

http://www.reviewboard.org/

Thanks

Michael

On 1/31/11 4:01 PM, Cedric Staub wrote:
> On Mon, Jan 31, 2011 at 03:53:01PM +0100, Michael Wechner wrote:
>> I understand, but one needs to review it anyway ;-)
>>
>> I don't like being the bottleneck, but in order to improve
>> this we need to introduce and need to stick to guidelines, such
>> that other people can review as well. If we cannot make this
>> happen and guarantee QA, then it won't change and
>> I won't go back to the situation where we end up with unreviewed
>> stuff leading us into misery.
>>
>> As discussed we are giving Mercurial a shot re patch management, but
>> this will only make it more efficient (hopefully) and it won't solve the
>> problem of QA itself
> I know, I didn't mean you shouldn't review the patch. I just wanted to
> draw your attention a bit ;-). Sorry if my comment was misleading.
>
> Cheers
> Cedric



More information about the Yanel-development mailing list