Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

gulp-imagemin error #28

Open
kirkbross opened this issue Mar 7, 2019 · 2 comments
Open

gulp-imagemin error #28

kirkbross opened this issue Mar 7, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@kirkbross
Copy link

kirkbross commented Mar 7, 2019

SOLVED: at the end of the buildImages function, I changed:

], done) : undefined

to:

], done()) : undefined


This project is awesome btw. I was having no problems and then suddenly it starting gulp-imagemin starting crashing npm start.

If I remove all the images from my src/img folder and then npm start, it runs fine. If I add the images back, or new new images while it's running, all is good. It's only if I CTRL+C stop and then npm start again that I get the below error.

I'm running Windows 10 with latest node and npm.

gulp-imagemin: Minified 22 images (saved 38.9 kB - 1.7%) [21:19:36] The following tasks did not complete: default, Booting Images Task: Build - Production Mode [21:19:36] Did you forget to signal async completion?

@IamManchanda
Copy link
Owner

Thanks will surely see this next weekend and other issues ...
Again thanks for making me figure out that this repo got love ... I had no idea honestly!

Apologies for that!

@IamManchanda IamManchanda added the bug Something isn't working label Mar 10, 2019
@emisrw
Copy link

emisrw commented Jul 29, 2019

I also have a problem with gulp-imagemin. It does not copy all images, only part. I would also consider adding a cache for photos and compress only new.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants