From 3cd5ecb9412c728669f9eee9f5e6360fd1fe4fc1 Mon Sep 17 00:00:00 2001 From: CheariX Date: Sun, 6 Oct 2024 00:42:37 +0200 Subject: [PATCH] fix: do not included downloaded assets in jekyll-minifier (#2749) If `download: true`, the site deployment fails. This caused e.g. issue #2548. I believe the issue appears because the 3rd party downloaded libs rely on ES6 Syntax, which jekyll-minifier cannot work on correctly. Also, I think we do not need to minify 3rd party downloaded libs at all. While this PR does **not** fix the issue above, it at least ensures that the site can be deployed with `download: true`. We still need better ES6 support as suggested in #2571. --- _config.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/_config.yml b/_config.yml index 4411534b7f60..b17f37e598a6 100644 --- a/_config.yml +++ b/_config.yml @@ -286,7 +286,7 @@ sass: # ----------------------------------------------------------------------------- jekyll-minifier: - exclude: ["robots.txt", "assets/js/search/*.js"] + exclude: ["robots.txt", "assets/js/search/*.js", "assets/libs/**/*"] uglifier_args: harmony: true