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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
