Include target branch in backport PR titles - #24868
Conversation
Backport PR titles now read `[Backport <branch>] <subject>` so the target release branch is visible in the PR list and notifications, not just in the base-ref column.
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: 3d8c9a8 | Docs | Datadog PR Page | Give us feedback! |
evalya-impact-summaryevalya impact analysis |
Validation ReportAll 21 validations passed. Show details
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3d8c9a898d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| new_branch=new_branch, | ||
| worktree_path=_worktree_path_for(app, new_branch), | ||
| pr_title=f'[Backport] {clean_subject}', | ||
| pr_title=f'[Backport {target_branch}] {clean_subject}', |
There was a problem hiding this comment.
Update the CLI help to match the generated title
When users run ddev release port-commit --help, the command still promises a PR titled [Backport] <subject> in port_commit.py:59, while this now generates [Backport <target-branch>] <subject>. Update that docstring so the documented user-visible behavior matches the PR that is actually created.
Useful? React with 👍 / 👎.
What does this PR do?
Backport PRs opened by
ddev release port-commitnow title as[Backport <branch>] <subject>instead of[Backport] <subject>.Motivation
Follow-up to #24470. With one merged PR fanning out to multiple release branches,
[Backport]alone made the PRs indistinguishable in the PR list and notifications; the target branch is now visible without opening each one.Review checklist (to be filled by reviewers)
qa/requiredif this PR needs QA validation, orqa/skip-qaif it does not. Exactly one of the two is required.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged