Skip to content

Commit 4c4fbf7

Browse files
CheariXOmidSa75
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 0a4f0b0 commit 4c4fbf7

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
@@ -286,7 +286,7 @@ sass:
286286
# -----------------------------------------------------------------------------
287287

288288
jekyll-minifier:
289-
exclude: ["robots.txt", "assets/js/search/*.js"]
289+
exclude: ["robots.txt", "assets/js/search/*.js", "assets/libs/**/*"]
290290
uglifier_args:
291291
harmony: true
292292

0 commit comments

Comments
 (0)