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

feat: upgrade gatsby-plugin-sharp to latest #3090

Closed

Conversation

jlengstorf
Copy link
Contributor

This should remove the vips warning output.

I've been using this with yarn link and it's been working properly without the console warnings. All tests pass with a couple snapshot updates. I think this should be good to go.

This should remove the `vips warning` output.
@gatsbybot
Copy link
Collaborator

Deploy preview ready!

Built with commit 2c219cc

https://deploy-preview-3090--gatsbygram.netlify.com

@gatsbybot
Copy link
Collaborator

Deploy preview ready!

Built with commit 2c219cc

https://deploy-preview-3090--using-drupal.netlify.com

@KyleAMathews
Copy link
Contributor

I haven't wanted to upgrade because of this issue lovell/sharp#832

Seems like too big of regression — I'd rather wait until libvips 8.6 & Sharp 0.19 are out.

Thoughts?

@jlengstorf
Copy link
Contributor Author

Ah. That makes sense. I haven't been using that feature, so didn't notice a regression. Do they have an ETA for 8.6/0.19? It's not a big deal, but those console logs are pretty noisy.

I'll close this in the meantime. (Though I'll probably keep using it locally. 😆)

@jlengstorf jlengstorf closed this Dec 3, 2017
@fk fk added API/Plugins status: blocked This issue/PR can't be solved at the moment and shouldn't be closed/merged labels Jan 19, 2018
@fk
Copy link
Contributor

fk commented Jan 26, 2018

@jlengstorf Looks like sharp v0.19 is here: #3675!

@jlengstorf jlengstorf deleted the feat/upgrade-sharp-version branch July 31, 2018 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: blocked This issue/PR can't be solved at the moment and shouldn't be closed/merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants