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:
SKARA-1587
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1399/files
- new: https://git.openjdk.org/skara/pull/1399/files/47dbf8d2..7abf5e32
Webrevs:
- 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