-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Sharp v0.23 removed overlayWith breaking gatsby-plugin-sharp #17162
Comments
Hi! Can you create a reproduction project for that and link it here? My personal site uses the versions you specified and my builds works fine https://github.com/LekoArts/portfolio/blob/cc0d70d118c0d06736fccec43afa88cf6b42ce59/yarn.lock#L6391-L6413 |
@LekoArts There isn't any need for repro, all information provided by @kevinastone are clear:
What maybe isn't clear from the description is that the problem occurs only if you actually use Deprecation warning from sharp v0.22: |
Should have added that I of course use duotone (https://github.com/LekoArts/portfolio/blob/cc0d70d118c0d06736fccec43afa88cf6b42ce59/src/templates/project.jsx#L172) in my site, wouldn't make sense to mention it otherwise. 😅 Hence we'd greatly appreciate a minimal reproduction or links to your repositories. |
Tests running the `gatsby-plugin-sharp` with duotone
Tests running the `gatsby-plugin-sharp` with duotone
Thanks for the repro, I'm also seeing this locally. |
#17392 has the fix (and some unit tests that would have caught this dependency api change). |
* Added failing test for #17162 Tests running the `gatsby-plugin-sharp` with duotone * Migrated overlayWith to composite `overlayWith` was removed in sharp `v0.23`. The replacement (`composite`) provides the superset functionality supporting an array of images to overlay/composite * Specify explcit blend: over mode for sharp composite
Published in |
…yjs#17392) * Added failing test for gatsbyjs#17162 Tests running the `gatsby-plugin-sharp` with duotone * Migrated overlayWith to composite `overlayWith` was removed in sharp `v0.23`. The replacement (`composite`) provides the superset functionality supporting an array of images to overlay/composite * Specify explcit blend: over mode for sharp composite
Description
v0.23
of Sharp (changelog) removed the deprecatedoverlayWith
which is still in the gatsby-plugin-sharp master:gatsby/packages/gatsby-plugin-sharp/src/duotone.js
Line 111 in 561d33e
gatsby-plugin-sharp
current master and current latest released2.2.18
declare sharp>=0.23 as a dependency:This causes duotone to fail when rendering graphql.
Steps to reproduce
Here's my graphql that is failing after upgrading gatsby:
Expected result
Render the graphql successfully.
Actual result
Environment
The text was updated successfully, but these errors were encountered: