[Sugar-devel] Usage of Signed-off-by / Acked-by - Re: Recent movement on Sugar master repository

[Daniel Narvaez]

On 4 April 2013 14:23, Gonzalo Odiard <gonzalo at laptop.org> wrote:

>
>
>
> About Reviewed-by, if you have say a 10 patches set, is one-by-one
>> interactive rebase the only way to add the tag to all the patches? I'm not
>> too attached to the one click pull request merging from the web UI...
>> pulling and pushing on the command line is not too complicated. Though if
>> you have to modify patches one by one it starts to get annoying...
>>
>>
> True. Modify every patch can be a pain, specially if we change the way we
> are working and move to use github web tools.
> I am not sure what is the best to do here. Web interface makes a little
> more difficult to see this header info.
> (Personally, I feel better in the command line yet)
>

I'm not sure the pull request workflow makes this worst though. It's very
easy to apply the patchset from the command line (just pulling a branch).
What is annoying is to modify the patches log one by one, but I suppose
that's the same with email workflow.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20130404/d0952038/attachment-0001.html>