RFR: 1587: Adding review comments should not mean approval status change

Zhao Song zsong at openjdk.org
Thu Oct 20 19:01:37 UTC 2022


On Thu, 20 Oct 2022 18:47:28 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> I don't think we should create a new `Review` here. I _think_ that the way to achieve the desired behavior is to ignore reviews where the verdict isn't `APPROVED` or `DISAPPROVED`. A `Review` with verdict `NONE` is essentially pointless when considering active reviews.

Ah, exactly! Thanks for the suggestion! I will change it.

-------------

PR: https://git.openjdk.org/skara/pull/1399


More information about the skara-dev mailing list