-
Notifications
You must be signed in to change notification settings - Fork 2
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
test: add a line #266
test: add a line #266
Conversation
/backport |
Backport failed for Please cherry-pick the changes locally. git fetch origin case4-target-branches
git worktree add -d .worktree/backport-266-to-case4-target-branches origin/case4-target-branches
cd .worktree/backport-266-to-case4-target-branches
git checkout -b backport-266-to-case4-target-branches
ancref=$(git merge-base 754e914ec3a9bdb7cbbc45a586074b5df66799f1 f2230426140cf8fc4c410f68d14a8410ce420ac5)
git cherry-pick -x $ancref..f2230426140cf8fc4c410f68d14a8410ce420ac5 |
So, the behavior is as before (the above ran without a /backport |
Successfully created backport PR for |
Okay, so that worked. Let's see how it runs with /backport |
Backport failed for Please cherry-pick the changes locally. git fetch origin case4-target-branches
git worktree add -d .worktree/backport-266-to-case4-target-branches origin/case4-target-branches
cd .worktree/backport-266-to-case4-target-branches
git checkout -b backport-266-to-case4-target-branches
ancref=$(git merge-base 754e914ec3a9bdb7cbbc45a586074b5df66799f1 f2230426140cf8fc4c410f68d14a8410ce420ac5)
git cherry-pick -x $ancref..f2230426140cf8fc4c410f68d14a8410ce420ac5 |
Okay, that did not work because the
So, default_mainlines of >1 are not supported at this time. It may be useful to add in the future, that would mean that we'd need to use the mainline setting only when cherry-picking merge commits explicitly. This can be added later while keeping behavior the same. |
Another idea is to allow specifying 0 as the mainline, where merge commits are skipped. This can also be supported later. |
This will test the default mainline functionality in https://github.com/korthout/backport-action/tree/341-default-mainline
See korthout/backport-action#341