Re: [PATCH 1/3] Prepare for non-interactive merge-preserving rebase
From: Stephen Haberman <hidden>
Date: 2016-06-15 22:45:25
Stephen, are you using this in production?
Kind of--I have not distributed a patched version of pull. But I have written test cases on our side and manually executing `GIT_EDITOR=: git rebase -i -p` works very well. Past occurrences aside, no one has needed to rebase a local merge yet.
How's it turning out?
I think it's great, but the primary problem will be getting devs to
actually remember to use it. E.g. I don't think they will type out:
git pull --rebase --preserve-rebase
Every time they pull. And they definitely don't do our current hack:
git fetch
GIT_EDITOR=: git rebase -i -p
I do have a wrapper shell script for people to use, but it hasn't seen
wide adoption yet. We have a draconian hook script that tries to
detect merges that should have been rebases and reject them, but
it's disabled for tweaking right now--when it gets turned back on,
I think more people will use the script.
In the long term, having "branch.name.preservemerges" and
"branch.autosetuppreservemerges" config options to parallel the
"branch.name.rebase" option and get us back to just "git pull"
would be great.
I've been meaning to submit patches for these two config options--I
figure I can hunt down how "branch.name.rebase" works and do the
appropriate copy/paste, but I haven't dedicated any time to it yet.
Thanks,
Stephen