On 14 February 2014 19:34, Gonzalo Odiard <gonz...@laptop.org> wrote:

>
>> * Backup
>>
>> This was posted a while ago but no one reviewed yet. Tests are missing.
>> Gonzalo, is it supposed to be otherwise complete? I feel we are risking to
>> miss the train here. Any reviewer volunteering to look at it? I'm probably
>> not going to be able to do that soon.
>>
>>
> I have not added test because never had a review.
> I understand is a big piece of code, but the pull request is more than 2
> months old :(
>
>
Yes that's bad. But how many patches did you review in the last 2 months? :)

Reviewing patches have to become a much higher priority for everyone or the
community will never grow and patches will keep being reviewed slowly. I'm
not picking on you, but that's the way it is.
_______________________________________________
Sugar-devel mailing list
Sugar-devel@lists.sugarlabs.org
http://lists.sugarlabs.org/listinfo/sugar-devel

Reply via email to