<div dir="ltr"><div><div><div>Hello,<br><br></div>we now have a list of reviewers but we still need to figure out a good review workflow.<br><br>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<br>
</div><br></div>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).<br><br><a href="https://github.com/sugarlabs">https://github.com/sugarlabs</a><br clear="all">
<div><div><div><div><br></div><div>If you are not familiar with it, see the official github documentation<br><br><a href="https://help.github.com/articles/using-pull-requests">https://help.github.com/articles/using-pull-requests</a><br>
</div><div><br></div><div>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<br>
<br><a href="https://help.github.com/articles/merging-a-pull-request">https://help.github.com/articles/merging-a-pull-request</a><br><br></div><div>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.<br>
<br></div><div>Let's post any question, doubt or complaint to the list, so that everyone can participate in the discussion.<br></div><div><br>-- <br>Daniel Narvaez<br>
</div></div></div></div></div>