[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