Skip to content

Detect merge strategy when cherry-picking PRs for backport#9518

Merged
hsbt merged 1 commit into
masterfrom
release-cherry-pick-rebase-merge
May 8, 2026
Merged

Detect merge strategy when cherry-picking PRs for backport#9518
hsbt merged 1 commit into
masterfrom
release-cherry-pick-rebase-merge

Conversation

@hsbt
Copy link
Copy Markdown
Member

@hsbt hsbt commented May 1, 2026

What was the end-user or developer problem that led to this PR?

The release task assumed every PR was merged with "Create a merge commit" and used git cherry-pick -m 1 MERGE_COMMIT_SHA.

So, I couldn't backport #5029 because this PR has merge commit.

What is your fix for the problem, implemented in this PR?

For PRs merged with "Rebase and merge" merge_commit_sha is just the tip of the rebased commits, so picking only that one silently drops the preceding commits and typically conflicts.

Inspect the parent count of merge_commit_sha and, when it is not a merge commit, fetch the PR commits from GitHub to decide between squash (single commit) and rebase (range of N commits).

Make sure the following tasks are checked

The release task assumed every PR was merged with "Create a merge
commit" and used `git cherry-pick -m 1 MERGE_COMMIT_SHA`. For PRs
merged with "Rebase and merge" `merge_commit_sha` is just the tip of
the rebased commits, so picking only that one silently drops the
preceding commits and typically conflicts. Inspect the parent count of
`merge_commit_sha` and, when it is not a merge commit, fetch the PR
commits from GitHub to decide between squash (single commit) and
rebase (range of N commits).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@hsbt hsbt merged commit fd5d360 into master May 8, 2026
94 checks passed
@hsbt hsbt deleted the release-cherry-pick-rebase-merge branch May 8, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant