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

[Thiago Macieira]

On segunda-feira, 12 de agosto de 2013 12:58:19, Josh Faust wrote:
> The Gerrit page says:
> "By default, changes required +2 for Code Review and +1 for Sanity Review
> categories to be approved"
> 
> That's what is required by the review system, but what is desired? If I've
> added multiple people to the review (or others have added themselves),
> should I wait for all (or most) of them to approve?

Unless it's controversial, or it's complex and you want to be absolutely sure, 
or one of the approvers asked for a second opinion, you can go ahead.

Usually, multiple approvals are required for changes touching multiple parts 
of the code, where no one single person feels comfortable with approving the 
whole lot. That's why I strongly advise that you make smaller changes, which 
can be approved by a single individual.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.qt-project.org/pipermail/development/attachments/20130812/f3c2db5e/attachment.sig>