[cr] RFR: 667: Webrev contents broken for rebased push

Robin Westberg rwestberg at openjdk.java.net
Thu Apr 15 14:15:50 UTC 2021


On Thu, 15 Apr 2021 13:54:31 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this patch that makes `webrev.js` calculate the "base"/"old" content from the "head"/"new" content and the reversed patch. This make `webrev.js` handle webrevs for force pushes. I also took the opportunity to reduce the number of network requests (by half) and also parallelized the initial three network requests.
> 
> Testing:
> - [x] Manual testing on a large number of webrevs
> 
> Thanks,
> Erik

Looks good!

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

Marked as reviewed by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/cr/pull/10


More information about the skara-dev mailing list