[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