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

Tomeu Vizoso tomeu at sugarlabs.org
Thu Sep 9 05:09:29 EDT 2010


On Wed, Sep 8, 2010 at 22:47, Daniel Drake <dsd at laptop.org> wrote:
> On 8 September 2010 14:44, Sascha Silbe
> <sascha-ml-reply-to-2010-3 at silbe.org> wrote:
>> 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.
>
> I think that's the safest bet, and will certainly avoid problems like this.

At least it would help if, when a review is moved to the mailing list,
a comment was added to the ticket about this. And of course, remove
the r? keyword.

Regards,

Tomeu

> Daniel
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel
>


More information about the Sugar-devel mailing list