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

Erik Helin ehelin at openjdk.java.net
Fri Apr 16 07:52:45 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

This pull request has now been integrated.

Changeset: 47665ae3
Author:    Erik Helin <ehelin at openjdk.org>
URL:       https://git.openjdk.java.net/cr/commit/47665ae3
Stats:     111 lines in 1 file changed: 56 ins; 43 del; 12 mod

667: Webrev contents broken for rebased push

Reviewed-by: rwestberg

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

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


More information about the skara-dev mailing list