florent/pages
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fix: do not included downloaded assets in jekyll-minifier (#2749)

Browse Source 
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.
This commit is contained in:
CheariX 2024-10-06 00:42:37 +02:00 committed by GitHub
parent b74b292cac
commit 421bdf860b
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
_config.yml
View File

@ -286,7 +286,7 @@ sass:
# ----------------------------------------------------------------------------- # -----------------------------------------------------------------------------
jekyll-minifier: jekyll-minifier:
exclude: ["robots.txt", "assets/js/search/*.js"] exclude: ["robots.txt", "assets/js/search/*.js", "assets/libs/**/*"]
uglifier_args: uglifier_args:
harmony: true harmony: true