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