On Tue, Mar 22, 2022 at 1:39 PM Wim Jongman <wim.jong...@gmail.com> wrote:

>
>
>> What particularly do you think does not work well with it?
>>
>
> File reviews or bound to the commit id and when you force push, the review
> is gone. See the screenshot below. This makes it very hard for the reviewer
> to see if their suggestions have been followed.
>

I don't have the same experience here in general: using force push shows a
line that allows to diff the previous and current version. On the review
page, the former review comments are still shown, and marked as "Outdated"
if the code changed in a way that the review is not easy to transfer to
newer patch.
_______________________________________________
platform-dev mailing list
platform-dev@eclipse.org
To unsubscribe from this list, visit 
https://www.eclipse.org/mailman/listinfo/platform-dev

Reply via email to