[Qgis-developer] forced update on the repo?
Richard Duivenvoorde
rdmailings at duif.net
Mon Jan 13 01:35:16 PST 2014
On 13-01-14 10:18, Matthias Kuhn wrote:
> Some general advice (althogh I'm not sure how exactly that happened) is to
>
> For creating a pull request
> ===============
> - always start a feature branch from current master
> - if you are coding a feature branch, don't "merge" anything in
> - before you create a pull request do "git fetch origin" and "git rebase
> origin/master" (given origin is the remote for upstream and not your own
> remote, check your .git/config or do "git remote -v | grep
> github.com/qgis")
> - you may do a "git rebase" like in the last line repeatedly without
> doing any damage (as long as the only purpose of your branch is to get
> merged into master)
> - Attention: After a rebase you need to "git push -f". CORE DEVS: IF YOU
> DO THIS ON THE UPSTREAM REPOSITORY YOU WILL BRING CAKE FOR EVERYBODY TO
> THE NEXT HF!
>
> For merging a pull request
> ===============
> Option A)
> - click the merge button (Creates a non-fast-forward merge)
>
> Option B)
> - Checkout the pull request (See
> https://gist.github.com/piscisaureus/3342247)
> - Test (Also required for option A, obviously)
> - checkout master, "git merge pr/1234"
> - Optional: "git pull --rebase": Creates a fast-forward, no "merge
> commit" is made. Cleaner history, but it is harder to revert the merge.
> - "git push"
>
> I hope that I got things right and that somebody finds this useful.
Hi Matthias,
I think it is VERY usefull. But I also think we should create a
'git'-page in documentation which contains this info.
I think it would also be helpfull to show the (more general) git
commands for creating a local branch etc.
I'll try to create a page for that with the information above, and add
some stuff that I think is usefull.
Plz have a look into it when finished.
Regards,
Richard Duivenvoorde
More information about the Qgis-developer
mailing list