RFR: 1856: Include a link to the webrev comment in the pull request's description
Zhao Song
zsong at openjdk.org
Wed Mar 29 22:03:40 UTC 2023
On Wed, 29 Mar 2023 21:58:06 GMT, Zhao Song <zsong at openjdk.org> wrote:
> Some users complained about that GitHub often hide the webrev comment and it's hard for them to find webrev comment among hundreds of comments.
>
> In this patch, the pr bot would add the link to webrev comment to the pr body.
>
> Originally, the comments from mlbridge bot would not trigger the update of the pr, now, when mlbridge bot **first** post the webrev comment, it will trigger the update of the pr.
bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckWorkItem.java line 100:
> 98: .collect(Collectors.joining());
> 99: commentString = commentString + comments.stream()
> 100: .filter(comment -> comment.author().username().equals("mlbridge[bot]"))
We also could compare `comment.author().id()` here, but I am not sure if it is a good idea to explicitly expose the id of mlbridge bot.
-------------
PR Review Comment: https://git.openjdk.org/skara/pull/1494#discussion_r1152525545
More information about the skara-dev
mailing list