<div dir="ltr"><div>So I'm not sure these are worth the overhead. Especially for  signing it doesn't seem to be very useful for us, if not in some special cases, and it adds one more thing we need to require from contributors... I'd rather bug them about improving the code then about using a tag of dubious utility.<br>
<br></div>Just my feeling really, I'm happy to keep using the tags if we think they are necessary. Just trying to streamline things a bit if possible.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On 2 April 2013 19:25, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Daniel,<br>
<br>
2013/4/2 Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com">dwnarvaez@gmail.com</a>>:<br>
<div class="im">> Hey,<br>
><br>
> sorry, I forgot about those. Though I think it's worth to think if we really<br>
> need them while we are reworking reviews a bit.<br>
><br>
> When is signed-off-by useful in sugar?<br>
<br>
</div>I think we just have been following other projects like the linux<br>
kernel.  In many projects this means "I certify that my code is<br>
compatible with the license of this project".  For me, it is useful<br>
only if the commiter is another user, to track the real author.<br>
<br>
<a href="http://wiki.sugarlabs.org/go/Development_Team/Code_Review" target="_blank">http://wiki.sugarlabs.org/go/Development_Team/Code_Review</a><br>
<div class="im"><br>
> How is reviewed-by applied? Is it the reviewer which rebase and add them<br>
> before pushing?<br>
<br>
</div>Yes we have been amending the patches before pushing, all manual work.<br>
 For me, the Acked-by signing has been useful to track who reviewed<br>
what.  But maybe not a big deal?<br>
<br>
So there are projects that do signing, and projects that don't.  I<br>
searched quickly for a way to add Acked-by when github merges a pull<br>
request, but haven't found anything.<br>
<div><div class="h5"><br>
> On 2 April 2013 18:30, Manuel Quiñones <<a href="mailto:manuq@laptop.org">manuq@laptop.org</a>> wrote:<br>
>><br>
>> Hi devs,<br>
>><br>
>> I've seen some movement in current master.  I suppose all this was<br>
>> reviewed, but what I wonder is if we are sticking to the signed-off-by<br>
>> / acked-by signing.<br>
>><br>
>> The commits in question are:<br>
>><br>
>> commit f423ec21b4bf0d953a470a383cc801b61a087e98<br>
>> Author: Walter Bender <<a href="mailto:walter.bender@gmail.com">walter.bender@gmail.com</a>><br>
>> Date:   Sat Mar 30 16:06:00 2013 -0400<br>
>><br>
>>     Add comment box to expanded entry<br>
>><br>
>> commit 541af0166030a5f3b7b52bdc23d416dbf688b5e9<br>
>> Author: Walter Bender <<a href="mailto:walter.bender@gmail.com">walter.bender@gmail.com</a>><br>
>> Date:   Sat Mar 30 10:45:22 2013 -0400<br>
>><br>
>>     Add CommentView widget to expanded entry<br>
>><br>
>> commit 248a758875e5a02a53ae57980f4fe2f3bb0bb91e<br>
>> Author: Walter Bender <<a href="mailto:walter.bender@gmail.com">walter.bender@gmail.com</a>><br>
>> Date:   Thu Mar 28 15:26:17 2013 -0400<br>
>><br>
>>     Make a separate method for _write_entry so method can be reused<br>
>><br>
>> commit 1af4b5ec7f03ed43da1616df946fe936e8577e91<br>
>> Author: Walter Bender <<a href="mailto:walter.bender@gmail.com">walter.bender@gmail.com</a>><br>
>> Date:   Thu Mar 28 15:25:41 2013 -0400<br>
>><br>
>>     Pass a widget to _create_scrollable so method can be reused<br>
>><br>
>><br>
>> --<br>
>> .. manuq ..<br>
>> _______________________________________________<br>
>> Sugar-devel mailing list<br>
>> <a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
>> <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
><br>
><br>
><br>
><br>
> --<br>
> Daniel Narvaez<br>
<br>
<br>
<br>
</div></div>--<br>
.. manuq ..<br>
</blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div>