-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gulp does not update 'date modified' of dist-directory after changes in partials - (zurb-advanced-template) #8544
Comments
My workaround at the moment is using this additional gulp-plugin:
|
The |
gulp-touch is also what we use in production. This is a known (config) issue in Gulp. |
Should we implement |
How should we decide on this @ncoden? In our projects we use sometimes I think this should be part of (one of) the templates, not the framework itself. |
@ncoden what should we do here? |
I can't install the gulp-touch plugin. I get this error:
I don't know how to solve this. Can someone help me please? |
Hi @JessDelAngel, this is a different issue. Did you already try to delete the lockfile and install it again? |
Hi @DanielRuf, thank you so much for the quick response. Where can I find the lockfile? (I'm sorry, but I don't know much about this kind of things) |
Try |
Thank you so much @DanielRuf! |
Did this solve your issue @JessDelAngel? |
Yes, it did! Thanks @DanielRuf :) |
Also see ahmadnassri/node-har-validator#112 (comment) and ahmadnassri/node-har-validator#113 why the version was not found anymore. The author unpublished two versions without a good reason. |
So far we did not receive fue'rther reports. I close this issue for now. It is probably fixed in gulp. Please let us know if this is not the case for you. |
I've got a rather specific bug and it might actually be a bug from gulp, not foundation. But in any case, it would be great to have a workaround.
My Setup:
I ran
npm install
andnpm update
, too.How can we reproduce this bug?
foundation watch
in a zurb template, edit any _partial.scss from thesrc/scss/components
-directorysrc/layouts
What did you expect to happen?
app.css
updates indist
dist
, since their header and footer were updated by altering the default.html insrc
What happened instead?
src
, regardless in changes in any imported scss-partials. To force an update , it is necessary to make any change in app.scss (like opening it, add a space anywhere, and save).dist
do not get updated when you make any changes in the default.html insrc
.I remember that before updating to 6.2 and gulp#4.0, all files in
dist
got a new "date modified". I think it's great that this does not happen anymore, only changed files get updated.The problem is just that imports like scss-partials or the panini-layout are not registered as changes.
I no expert in gulp, but it seems to be connected to gulp-vinyl. I found these related discussions on gulp:
gulpjs/gulp#1414
gulpjs/gulp#1461
It seems it was resolved, though?:
gulpjs/vinyl#75
So I guess these bug reports were about the an older bug not updating the date at all, and not about not noticing imported changes.
Edit , here's the package.json:
The text was updated successfully, but these errors were encountered: