RFR: mlbridge: drop prefix "webrev" from webrev paths

Robin Westberg rwestberg at openjdk.java.net
Tue Aug 18 06:43:03 UTC 2020


On Mon, 17 Aug 2020 17:43:10 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> please review this patch that drops the "webrev" prefix from the last part of the path for URLs to automatically
> generated webrevs. For example, the current path looks like `https://webrevs.openjdk.java.net/skara/704/webrev.00`,
> they will now look like `https://webrevs.openjdk.java.net/skara/704/00`. This is partly done in preparation for
> uploading JSON versions of webrevs.  Testing:
> - [x] `make test` passes on Linux x64
> - [x] Updated a number of unit tests
> 
> Thanks,
> Erik

Looks good!

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

Marked as reviewed by rwestberg (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/718


More information about the skara-dev mailing list