<div dir="ltr"><div>Cool!<br><br></div>I suppose you are going to want that on the 0.98 branch too. I'm not sure what we should in those cases exactly... I mean merge it in github or in gitorious directly? No strong feeling here.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 April 2013 22:04, 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">
2013/4/3 Daniel Narvaez <<a href="mailto:dwnarvaez@gmail.com">dwnarvaez@gmail.com</a>>:<br>
<div><div class="h5">> Hello,<br>
><br>
> we now have a list of reviewers but we still need to figure out a good<br>
> review workflow.<br>
><br>
> I and Walter have been playing with github pull requests in the past few<br>
> days. Let's try to extend the experiment to everyone and test if it really<br>
> works well for us. Essentially this means a couple of things<br>
><br>
> 1. Patch contributors should submit their code using pull requests. I setup<br>
> forks of the Glucose repositories here (I will take care of syncing back to<br>
> gitorious).<br>
><br>
> <a href="https://github.com/sugarlabs" target="_blank">https://github.com/sugarlabs</a><br>
><br>
> If you are not familiar with it, see the official github documentation<br>
><br>
> <a href="https://help.github.com/articles/using-pull-requests" target="_blank">https://help.github.com/articles/using-pull-requests</a><br>
><br>
> 2. Reviewers will get email notifications when new patches are posted.<br>
> Please use the web interface to comment on the changes and request fixes.<br>
> When you are happy with the patch, feel free to merge it. Some info on how<br>
> to do it<br>
><br>
> <a href="https://help.github.com/articles/merging-a-pull-request" target="_blank">https://help.github.com/articles/merging-a-pull-request</a><br>
><br>
> There has been some discussion on the list about the details of the<br>
> workflow, but really I would like everyone to try things out before<br>
> documenting the process more precisely. Similarly I'm planning to write a<br>
> checklist reviewers should follow but I'd like that to emerge from the<br>
> experiment.<br>
><br>
> Let's post any question, doubt or complaint to the list, so that everyone<br>
> can participate in the discussion.<br>
<br>
</div></div>Well done!  I have a patch in the workings for:<br>
<a href="http://bugs.sugarlabs.org/ticket/3819" target="_blank">http://bugs.sugarlabs.org/ticket/3819</a><br>
<br>
It is a bugfix that brings back the drag and drop to the clipboard<br>
inside the frame.  Will do the pull request when I finish.<br>
<br>
.. manuq ..<br>
</blockquote></div><br><br clear="all"><br>-- <br>Daniel Narvaez<br>
</div>