Skip to content
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

Fix gulp imagemin options #1839

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Conversation

gnowland
Copy link

Image minification options (specifically SVG plugin options) were not
working when using the latest version of gulp-imagemin due to a new
options passing structure, see:
https://github.com/sindresorhus/gulp-imagemin/releases/tag/v3.0.0

Image minification options (specifically SVG plugin options) were not
working when using the latest version of gulp-imagemin due to a new
options passing structure, see:
https://github.com/sindresorhus/gulp-imagemin/releases/tag/v3.0.0
@gnowland gnowland changed the base branch from master to sage-8 February 21, 2017 16:17
@retlehs retlehs merged commit e63d947 into roots:sage-8 Feb 21, 2017
@retlehs
Copy link
Sponsor Member

retlehs commented Feb 21, 2017

thanks!

@gnowland gnowland deleted the fix-gulp-imagemin-options branch February 21, 2017 16:57
asuh added a commit to asuh/sage-starter that referenced this pull request Feb 22, 2017
Latest version of gulp-imagemin breaks with old method
roots/sage#1839
smutek pushed a commit to ainsleyagency/sage842 that referenced this pull request Mar 7, 2017
Update dependencies to bring theme up to date with Sage 8.5 (minus
Bootstrap 4).

Should enable use of Node versions >= v6 and eliminate
issues with deployments via deploybot

* Update gulp-imagemin options
* ([roots#1839](roots#1839))
* Add error handling to `scripts` and `styles` gulp tasks
* ([roots#1832](roots#1832))
* Update dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants