[Sugar-devel] Code review (was: Re: [PATCH] Added Protected-Activities-Support to sugar (SL#2087))

Sascha Silbe sascha-ml-reply-to-2010-3 at silbe.org
Wed Sep 8 16:44:21 EDT 2010


Excerpts from Daniel Drake's message of Wed Sep 08 21:34:33 +0200 2010:

> Where a review process has started on trac, please keep the patch
> reviews on trac.

That's probably my fault, I've been referring people from Trac to the
mailing list for review. My assumption was that they're simply not
aware of the recent "experimental" changes to the Code Review Process
(that are not reflected on the corresponding wiki page, only on the
talk page), but would have liked to benefit from the new, streamlined
process.

If there's consensus that we want to stay with the method first chosen
by the submitter, then I'll refrain from doing that in the future and
will concentrate on patches submitted on the mailing list.

Sascha

--
http://sascha.silbe.org/
http://www.infra-silbe.de/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
Url : http://lists.sugarlabs.org/archive/sugar-devel/attachments/20100908/071e114f/attachment.pgp 


More information about the Sugar-devel mailing list