When reusing PR, update the title and body#64
When reusing PR, update the title and body#64jinfwhuang wants to merge 1 commit intofasterci:mainfrom
Conversation
![codereviewbot-ai[bot]](https://avatars.githubusercontent.com/in/309638?s=60&v=4){kind=small}
| } | ||
|
|
||
| // Update the first matching PR | ||
| existingPR := prs[0] |
There was a problem hiding this comment.
The logic in updateExistingPR always updates the first PR returned (prs[0]). If multiple open PRs match the criteria, this may result in updating the wrong PR. Consider iterating through all matching PRs or adding additional checks to ensure the correct PR is updated.
|
hey @jinfwhuang sorry for the delay. mind elaborating a bit on the intent of this PR? the way im thinking about it (in the course of normal operations): a commit on main produces rendered outputs associated with the release train, a commit with the rendered output is pushed to the release branch (the message of this commit acts as a back-pointer to the commit which yielded the changes, and the deploy PR description is updated with the link to the PR for this same commit). |
2 participants
When we are using a PR, we should be able to update the PR title and body. This is especially helpful when we want to include the commit information as part of the PR.