git go-patch extract
is sensitive to am.threeWay
configuration, making patch resolution incompatible with CI in some cases
#1233
Labels
To make it easier to work on patches, I tried setting
git config --global am.threeWay true
. This means when I rungit go-patch apply
, it will automatically do 3-way merges, rather than me needing to rungit am -3
myself to see if that works.The problem is that
git go-patch extract
usesam
to detect whether a patch has changes and needs to be re-extracted. So, 3-way merge resolutions won't get checked into the dev's PR. CI doesn't usegit am -3
, so it fails to apply the patches. Noticed with #1232.Unsetting
am.threeWay
for theam
used duringgit go-patch extract
should be enough to fix this.Workaround:
git config --global am.threeWay false
.Intentionally setting
am.threeWay=true
duringgit go-patch apply
might be nice, however this could be confusing, because it might not be obvious that some conflicts were resolved. I think this can wait for #542.The text was updated successfully, but these errors were encountered: