[Development] Proposal: Allow contributors to +1 sanity review.

Thiago Macieira thiago.macieira at intel.com
Tue Aug 13 20:06:39 CEST 2013


On terça-feira, 13 de agosto de 2013 10:51:17, Alan Alpert wrote:
> I run into it fairly often.
> https://codereview.qt-project.org/#q,reviewer:aalpert%2540blackberry.com+sta
> tus:merged,n,z shows five times this month that it needed to be overridden
> in my
> reviews.
> 
> > When it does, we can override it.
> > That's why we can override it in the first place: because it sometimes
> > makes mistakes.
> 
> Agreed. I just don't think that process is as smooth as it could be.

Ok, looks like I should have read the *subject* of the email.

Yes, allow anyone to +1 sanity review has +1 from me.

-- 
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/20130813/f06fa96f/attachment.sig>


More information about the Development mailing list