RFR: 2331: Improve review notes [v3]

[Pavel Rappo]

On Fri, 2 Aug 2024 19:31:00 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> @kevinrushforth ping
>
> Sorry for the delay.
> 
> For the (now largely unused) case of `useStaleReviews=true`, I would expect no change from today's behavior: Regardless of whether or not the review criteria is met, print "⚠️ Review applies to HASH" for all stale reviews.
> 
> For the (now widely-used) case of `useStaleReviews=false`: If the review criteria is _not_ met, print "🔄 Re-review required (review applies to HASH)" for all stale reviews; if the review criteria _is_ met, print "Review applies to HASH" with no emoji for all stale reviews.

To further clarify, the algorithm you propose applies to the usual case where some commits have been added on top of those reviewed; and not to the more rare cases, where either the reviewed commit is no longer present or the PR has changed its target branch. Is that correct?

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

PR Review Comment: https://git.openjdk.org/skara/pull/1679#discussion_r1705587719