Skip to content

Commit 2fbbb0e

Browse files
CheariXodcambc
authored andcommitted
fix: do not included downloaded assets in jekyll-minifier (alshedivat#2749)
If `download: true`, the site deployment fails. This caused e.g. issue alshedivat#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 alshedivat#2571.
1 parent 8f064a3 commit 2fbbb0e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

_config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ sass:
298298
# -----------------------------------------------------------------------------
299299

300300
jekyll-minifier:
301-
exclude: ["robots.txt", "assets/js/search/*.js"]
301+
exclude: ["robots.txt", "assets/js/search/*.js", "assets/libs/**/*"]
302302
uglifier_args:
303303
harmony: true
304304

0 commit comments

Comments
 (0)