If they’ve edited a tracked file that has been modified since their last update, they’ll get the same conflict with rebasing that they would get with pull.
Rebasing will change the order of the two relevant commits, but I don’t think it will change the fact that there’s a conflict.
I would advise them (as non-contributors) either to change a tracked file or to add it to the exclude list in .git/info/exclude. Then they won’t have any conflicts no matter what they do.
-
MODX Staff
- 10,725 Posts
Sorry, I didn’t really mean to say conflicts; that is unavoidable and goes with the territory of changing core files, in any RCS. What rebasing vs. pulling will avoid is diverging from the upstream history, which can get really confusing and messy. You should always replay your changes on top of the latest upstream snapshot, which is what rebasing does. This keeps your history simple, clean, and in-sync with the project.
Avoiding conflicts by excluding things is not good IMO; that could easily lead to otherwise avoidable problems.
Good points. What do you think about having them alter the config file so git pull will automatically fetch and rebase in a single step?
It’s definitely doable, but I’m not sure it’s supported in all git clients.
git pull --rebase is another one-step option that I’m pretty sure will work on any client.