Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

set modified time when modifying file #709

Closed
wants to merge 1 commit into from
Closed

set modified time when modifying file #709

wants to merge 1 commit into from

Conversation

WraithKenny
Copy link
Contributor

Gulp plugins that create/modify output files are responsible for updating file modified timestamps as of Gulp 4 gulpjs/gulp#2193 (comment)

This change simply updates the file stat appropriately.

Fixes #706

Gulp plugins that create/modify output files are responsible for updating file modified timestamps as of Gulp 4 gulpjs/gulp#2193 (comment)

This change simply updates the file stat appropriately.

Fixes #706
@WraithKenny
Copy link
Contributor Author

Welp, it fails the tests, but the idea is right.

@dryoma
Copy link

dryoma commented Apr 20, 2019

Would be nice if this actually got merged ;)

@jaap3
Copy link

jaap3 commented Apr 21, 2020

A colleague and I just spent 2.5 hours debugging an issue that was ultimately caused by gulp-sass not updating the mtime of the output files. It would be really nice if this was merged (after fixing the linting issue I guess).

@WraithKenny
Copy link
Contributor Author

Either of you know how to fix this so the tests pass?

@WraithKenny
Copy link
Contributor Author

Close in favor of #763

@xzyfer
Copy link
Collaborator

xzyfer commented Apr 23, 2020

Released in v4.1.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not updating modified timestamp of output when working with @import
4 participants