[Development] Review process: when should I merge to staging?

[Laszlo Papp]

On Mon, Aug 12, 2013 at 9:42 PM, Josh Faust <jfaust at suitabletech.com> wrote:

>
>
>> Thiago covered it expertly, but I'd also suggest giving it some more
>> time if specific other reviewers have expressed interest in it (such
>> as by repeated comments). In a case like this one, where it has been
>> entirely a dialogue with the person who +2'd it, no waiting period
>> seems necessary.
>>
>
> To be clear: in the reviewer table, a checkmark is +2/approved? I've  seen
> +1s in the gerrit UI but never a +2.
>

Yes.


> I noticed this morning that I forgot to +1 sanity review when I
>> approved it. If the sanity bot comments have been covered in the
>> discussion already, like in that patch, just go ahead and +1 sanity
>> review it yourself if it has +2 code review. The -1 ratings from the
>> sanity bot do not require a separate approver to override, it's just
>> raising a red flag that should be addressed in the review before it's
>> given +2.
>>
>
> I've never seen the sanity review options -- do they only show up if the
> Sanity Bot -1'd it?
>

No.


> Or do I just merge without Sanity Bot approval?
>

It needs to be overriden if it is a false positive.

-- Laszlo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/development/attachments/20130812/45ca0e5b/attachment.html>