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

Checkout draft branch instead of target in suggestion #423

Merged
merged 1 commit into from
May 28, 2024

Conversation

korthout
Copy link
Owner

When resolving the conflicts of a draft backport pull request, the instructions say to checkout the target branch in a new worktree. However, that doesn't make sense for the draft pull request, because we've already created a backport branch. We should checkout the backport branch (branchname) instead of target.

resolves #421

When resolving the conflicts of a draft backport pull request, the
instructions say to checkout the target branch in a new worktree.
However, that doesn't make sense for the draft pull request, because
we've already created a backport branch. We should checkout the backport
branch (branchname) instead of target.
@korthout korthout merged commit 9237785 into main May 28, 2024
1 check passed
@korthout korthout deleted the korthout-fix-draft-checkout branch May 28, 2024 11:39
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.

Conflict resolution draft_commit_conflicts provides incorrect instructions
1 participant