[Sugar-devel] Reviews experiment

[Daniel Narvaez]

Hello,

we now have a list of reviewers but we still need to figure out a good
review workflow.

I and Walter have been playing with github pull requests in the past few
days. Let's try to extend the experiment to everyone and test if it really
works well for us. Essentially this means a couple of things

1. Patch contributors should submit their code using pull requests. I setup
forks of the Glucose repositories here (I will take care of syncing back to
gitorious).

https://github.com/sugarlabs

If you are not familiar with it, see the official github documentation

https://help.github.com/articles/using-pull-requests

2. Reviewers will get email notifications when new patches are posted.
Please use the web interface to comment on the changes and request fixes.
When you are happy with the patch, feel free to merge it. Some info on how
to do it

https://help.github.com/articles/merging-a-pull-request

There has been some discussion on the list about the details of the
workflow, but really I would like everyone to try things out before
documenting the process more precisely. Similarly I'm planning to write a
checklist reviewers should follow but I'd like that to emerge from the
experiment.

Let's post any question, doubt or complaint to the list, so that everyone
can participate in the discussion.

-- 
Daniel Narvaez
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20130403/33cebd99/attachment.html>