Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document shell_command_after to merge diffs #262

Merged
merged 1 commit into from
Dec 16, 2019
Merged

Conversation

yajo
Copy link
Contributor

@yajo yajo commented Dec 16, 2019

This is a useful case for Doodba users, where possibly you want to pre-merge a patch done for Odoo v11 into your Odoo v13 deployment, before Odoo approves it.

Using just git branches is not much helpful in this case because no git history is shared, so conflicts would arise always.

In fact, this might be the best way to apply merges always. That's to be investigated.

Upstream support for diffs and patches is being tracked in acsone/git-aggregator#34, but in the mean time this option is available today, so it's better to make it more obvious.

@Tecnativa TT19951

This is a useful case for Doodba users, where possibly you want to pre-merge a patch done for Odoo v11 into your Odoo v13 deployment, before Odoo approves it.

Using just git branches is not much helpful in this case because no git history is shared, so conflicts would arise always.

In fact, this might be the best way to apply merges *always*. That's to be investigated.

Upstream support for diffs and patches is being tracked in acsone/git-aggregator#34, but in the mean time this option is available today, so it's better to make it more obvious.

@Tecnativa TT19951
@yajo yajo self-assigned this Dec 16, 2019
@yajo yajo requested a review from pedrobaeza December 16, 2019 09:36
@yajo yajo merged commit 964fc15 into master Dec 16, 2019
@yajo yajo deleted the doc-shell_command_after branch December 16, 2019 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants