Skip to content
This repository has been archived by the owner on Sep 15, 2023. It is now read-only.

watch task will not compile more than 48 html files #561

Open
markjohnson4 opened this issue Aug 17, 2018 · 5 comments
Open

watch task will not compile more than 48 html files #561

markjohnson4 opened this issue Aug 17, 2018 · 5 comments
Assignees

Comments

@markjohnson4
Copy link

I just confirmed this to be the case with a fresh installation of blendid. This can easily be recreated by creating 50 copies of the index.html file (with different names), running "yarn run blendid", deleting production html files, and saving an html file in the "src/html" folder to trigger re-compiling the html. You'll see that only 48 files get compiled. No more.

@kamilczujowski
Copy link

Same Problem here, any idea?

@olets
Copy link
Contributor

olets commented May 28, 2019

@markjohnson4 sorry for the slow response!!

I am not able to replicate to 50-file aspect of this.

I do however see a bug that might be what's going on for you: the html watch task appears to only notice changes to files that existed when you ran yarn run blendid.

If you re-save one of the original files (after creating a new one, with or without deleting two, and regardless of file count) does the new file show up in your destination directory?

@olets olets self-assigned this May 28, 2019
@olets
Copy link
Contributor

olets commented May 28, 2019

html task issue is now captured in #580. Thanks for calling our attention to it 👍

@kasparsz
Copy link

kasparsz commented Jun 4, 2019

I managed to replicate on macOS, issue is related to gulp + browserSync BrowserSync/gulp-browser-sync#79

In my case I had more than 48 html files, fix seems to be to comment out in html.js
.pipe(browserSync.stream())

Downside of course is that browser doesn't reloads on html changes anymore, but at least more than 48 files can be created.

@olets
Copy link
Contributor

olets commented Jun 4, 2019

@kasparsz thanks for finding that issue. The workaround might be to refactor Blendid and trigger the reload manually. Suppose we could also do something with gulp-count but that would add complexity.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants