digitalsparky / jekyll-minifier

Jekyll HTML/XML/CSS/JS Minifier utilising yui-compressor, and htmlcompressor
GNU General Public License v3.0
245 stars 25 forks source link

Exclusions do not appear to be working #26

Closed edelstone closed 7 years ago

edelstone commented 7 years ago

I'm not a pro at this stuff, but exclusions of any sort (e.g. index.html, *.html, *.css and others) do not appear to be working in a basic Jekyll 3.5.0 build.

Here is an example of the code I'm putting in config.yml

jekyll-minifier:
  exclude: '*.css'
jp26jp commented 7 years ago

Agreed. This started happening to me after Jekyll 3.5 as well.

edelstone commented 7 years ago

Looks like there's a PR open about this. #24.

digitalsparky commented 7 years ago

24 has now been merged and a new version published. Apologies for taking so long, moving house is a big task ;).

edelstone commented 7 years ago

Thank you and good luck with the move!

sonnym commented 7 years ago

I was still having issues with exclude not working (technically, it worked only for the first file encountered), and have opened this pull request: https://github.com/digitalsparky/jekyll-minifier/pull/28

edelstone commented 7 years ago

Good looking out @sonnym. I haven't had a chance to check the fix that got pushed.

komplexb commented 7 years ago

Can't tell if exclude is working on my Jekyll 3.2.1 installation.

When I use the snippet below these files still get modified. Is my format correct?

jekyll-minifier:
  exclude: ['.htaccess', '.php']
  preserve_php: true
sonnym commented 7 years ago

Looks like the jekyll version was locked to 3.5 before I wrote the exclusion fix, so the version of jekyll-minifier you are using would be before it was merged: https://github.com/digitalsparky/jekyll-minifier/blame/master/jekyll-minifier.gemspec#L29

Also, even if your jekyll version were up to date, however, the fix has not been released, so you would need to install from github.

angel-carvajal commented 1 year ago

this is still a problem on latest version. can we re-open this?