[Sugar-devel] Github review workflow

[Manuel Quiñones]

2013/3/28 Daniel Narvaez <dwnarvaez at gmail.com>:
> Hi,
>
> I started experimenting a bit with github code reviews, with Walter as
> ginuea pig :) We wasn't too sure about stuff like rebasing, using
> separate branches etc, so tonight I played a bit myself with creating
> pull requests.
>
> Something like the following might be a decent start for a workflow
>
> 1 Create one branch per topic

0. Fork

I forgot to do it in my first pull-request submission.

>
> git checkout -b topic1
>
> 2 Make one or more commits
> 3 Push the branch
>
> git push origin topic1
>
> 4 Submit a pull request for the branch (web UI)
>
> 5a The reviewer merges the patch.
> 5b The reviewer rejects the patch (and closes the request).
> 5c The reviewer requires changes (and closes the request).
>
> If 5c:
>
> 6 Make changes using interactive rebase
> (http://git-scm.com/book/en/Git-Tools-Rewriting-History#Changing-Multiple-Commit-Messages)
>
> git rebase -i master
>
> 7 Push the changes to another remote branch
>
> git push origin topic1:topic1-try2
>
> 8 Submit the new pull request (web UI)
>
>
> If someone has experience with github suggestions would be welcome.
> Otherwise I hope Walter will keep being the ginuea pig in this
> experiment  :)
>
> --
> Daniel Narvaez
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel



--
.. manuq ..