RFR: 1925: When archiving a comment, mlbridgeBot would strip everything after the first command
Erik Joelsson
erikj at openjdk.org
Fri Jun 2 16:30:32 UTC 2023
On Fri, 2 Jun 2023 16:08:31 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> I'm not sure about this. We already send email when someone approves a PR. I don't think we need to send emails with an empty review.
>
> According to the previous logic, for example, if we approve a PR and the review body only contains command like "/reviewers 2", then this review will be ignored. This issue is what I want to resolve.
>
> But now I also think something in my logic is wrong.
See this PR that I reviewed this morning: https://github.com/openjdk/jdk/pull/14261
I didn't put any text in the review comment/body, but we still had this email sent: https://mail.openjdk.org/pipermail/build-dev/2023-June/039655.html
-------------
PR Review Comment: https://git.openjdk.org/skara/pull/1524#discussion_r1214583476
More information about the skara-dev
mailing list