[cr] Integrated: index.html: inline js
Erik Helin
ehelin at openjdk.java.net
Fri Apr 16 08:54:49 UTC 2021
On Fri, 16 Apr 2021 08:46:34 GMT, Erik Helin <ehelin at openjdk.org> wrote:
> Hi all,
>
> please review this patch that inlines `webrev.js` into `index.html` to avoid an additional network request. It would have been better to keep the JS in a separate file if we could:
>
> - minify the JS
> - configure cache control headers
>
> GitHub Pages unfortunately does not allow any of the above (and we don't want a build step for minification), therefore the best option is to inline the JS into `index.html`.
>
> Thanks,
> Erik
This pull request has now been integrated.
Changeset: 4ab6f42e
Author: Erik Helin <ehelin at openjdk.org>
URL: https://git.openjdk.java.net/cr/commit/4ab6f42e
Stats: 3034 lines in 2 files changed: 1506 ins; 1528 del; 0 mod
index.html: inline js
Reviewed-by: rwestberg
-------------
PR: https://git.openjdk.java.net/cr/pull/13
More information about the skara-dev
mailing list