search

ixkaito / frasco

Quick starter for Jekyll including full setup for Sass, PostCSS, Autoprefixer, stylelint, TypeScript, Webpack, ESLint, imagemin, Browsersync, etc.
https://ixkaito.github.io/frasco/
MIT License
131 stars 20 forks source link

Watch.js not watching for changes in html collections. #37

Closed mckeever02 closed 5 years ago

mckeever02 commented 5 years ago

Hey,

I just ran into an issue where I couldn't get change to compile for a collection consisting of html pages e.g. _account > index.html.

I fixed this by adding './**/*.html', to watch.js.

Shouldn't this be added by default or am I missing something?

Great plugin btw!

timpotter commented 5 years ago

Yeah, I've ran into the same issue before and have used the same solution as @mckeever02

ixkaito commented 5 years ago

@mckeever02 @timpotter Thank you both! You're right. ./**/*.html is better. On older versions, Browsersync reloads too many times if you set ./**/*.html. This issue seems to be improved.

I changed *.html to ./**/*.html and removed config.jekyll.includes + '/**/*', config.jekyll.layouts + '/**/*', config.jekyll.posts + '/**/*' on v0.8.8.