From 9d584e68660b388d43f0f9ea0c31caae03410530 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jeremiah=20Men=C3=A9trey?= Date: Sat, 7 Aug 2021 02:37:05 +0200 Subject: [PATCH] Upgrade rollup pipeline & fix minification The closure compiler has been replaced by terser for the minification step, as closure does not handle correctly variables (or in this case constants) declared with the same name as some globals (in this case `document`). This is a valid error for scripts, but in ES modules this kind of redeclarations do not cause problems (at least for `document`). For details see: - https://github.com/ampproject/rollup-plugin-closure-compiler/issues/92 - https://github.com/google/closure-compiler/issues/3098 - https://github.com/google/closure-compiler/issues/2471 Also terser is way faster than closure, and produces a slightly smaller minified script. --- demo/footnotes.htm | 2 +- demo/index.htm | 2 +- package-lock.json | 975 ++++++++++++++++++--------------------------- package.json | 12 +- rollup.config.js | 6 +- 5 files changed, 391 insertions(+), 606 deletions(-) diff --git a/demo/footnotes.htm b/demo/footnotes.htm index f2fa771..5f0b35b 100644 --- a/demo/footnotes.htm +++ b/demo/footnotes.htm @@ -27,7 +27,7 @@ }