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

Zhao Song zsong at openjdk.org
Thu Oct 20 19:21:47 UTC 2022

> The issue here is ' if a user leaves additional comments using the "Review changes" dialog, and selects the "Comments", then their approval is considered revoked.'
> I believe this happens because if we use "Review changes" to add a comment, it will add a review with `Review.Verdict.NONE` to our pr.  According to our current logic, this new review will cover old reviews, so the approval will be revoked.
> I made changes to `CheckablePullRequest#filterActiveReviews`. Now If latest review is just a comment, the verdict will inherit from the previous review.

Zhao Song has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:



  - all: https://git.openjdk.org/skara/pull/1399/files
  - new: https://git.openjdk.org/skara/pull/1399/files/47dbf8d2..7abf5e32

 - full: https://webrevs.openjdk.org/?repo=skara&pr=1399&range=01
 - incr: https://webrevs.openjdk.org/?repo=skara&pr=1399&range=00-01

  Stats: 12 lines in 1 file changed: 0 ins; 10 del; 2 mod
  Patch: https://git.openjdk.org/skara/pull/1399.diff
  Fetch: git fetch https://git.openjdk.org/skara pull/1399/head:pull/1399

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

More information about the skara-dev mailing list