RFR: 165: Replace large webrev files with placeholders instead of dropping them
Robin Westberg
rwestberg at openjdk.java.net
Thu Dec 5 07:43:00 UTC 2019
On Wed, 4 Dec 2019 15:47:30 GMT, Erik Helin <ehelin at openjdk.org> wrote:
>> Hi all,
>>
>> Please review this change that replaces large files in published webrevs with a placeholder message, instead of just omitting them.
>>
>> Best regards,
>> Robin
>
> This looks good in general, but won't this destroy the navigation arrows for e.g. the udiff or frames view? Or maybe it just overwrites the content, I don't recall
It would indeed destroy navigation, I've updated it a bit to handle .html files a bit nicer. This also ensures that the instructions are displayed with proper line breaks as they will be placed inside `<pre>` tags if possible.
-------------
PR: https://git.openjdk.java.net/skara/pull/292
More information about the skara-dev
mailing list