RFR: 2292: Improve the labels of Review/Commit links for backport branches
Zhao Song
zsong at openjdk.org
Fri Jul 5 22:38:56 UTC 2024
On Fri, 5 Jul 2024 22:23:48 GMT, Zhao Song <zsong at openjdk.org> wrote:
> The patch is trying to add a branch name to the commit link, review link, commit comment, review comment in JBS.
>
> Adding a branch name for commit link and comment is feasible and easy, but adding a branch name for review link and review comment is not that easy because people can retarget pull request. Therefore, the bot needs to save the target branch name in the storage.
>
> I believe my patch will be compatible with the existing storage files.
bots/notify/src/main/java/org/openjdk/skara/bots/notify/PullRequestWorkItem.java line 140:
> 138: if (pr.targetBranch() != null) {
> 139: ret.put("targetBranch", JSON.of(pr.targetBranch()));
> 140: }
We have two options for handling non-replaced entries:
1: Do not add a null JSON for targetBranch
2: Add a null JSON for targetBranch
Currently, I chose option 1
-------------
PR Review Comment: https://git.openjdk.org/skara/pull/1671#discussion_r1667191910
More information about the skara-dev
mailing list