-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Gatsby v4 upgrade [Fixes #4271] #4324
Conversation
upgrades gatsby-plugin-sharp and gatsby-transformer-sharp
Removes PRESERVE_WEBPACK_CACHE from gatsby config per Gatsby logs: ("Those flags no longer have any effect and you can remove them from config")
gatsby@4.0.2, gatsby-plugin-sharp@4.0.1, react-instantsearch-dom@6.15.0
@wackerow I've changed the config for I've used the default configs from the plugin. This means that the
Things to consider:
options: {
output: `/`, // the default value is `/sitemap`
...
}
|
This reverts commit cf830aa.
These parameters are ignored on Gatsby images marked as FULL_WIDTH, causing console warnings when left in.
This catch is not really an "error", as it is expected to occur on certain files and can default to returning true. Removing should help tech debt by reducing number of unhelpful errors/warnings in build log.
Fixes: The requested width "100px" for a resolutions field for the file /usr/src/app/www/src/assets/dev-tools/nftschool.png was larger than the actual image width of 90px! If possible, replace the current image with a larger one.
@@ -76,7 +80,6 @@ const checkIsMdxOutdated = (path) => { | |||
intlMatch += match.replace(re, (_, p1, p2) => p1 + p2) | |||
}) | |||
} catch { | |||
console.error("regex error") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we still want this error log.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Took this out after it was just producing a handful of non-specific errors in the build console. Will replace it with
console.error(`regex error in ${englishPath}`)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After chatting with @pettinarip decided to change this to a console.warn
instead
src/content/developers/tutorials/uniswap-v2-annotated-code/index.md
Outdated
Show resolved
Hide resolved
src/content/developers/tutorials/uniswap-v2-annotated-code/index.md
Outdated
Show resolved
Hide resolved
removes `cp` command from onPostBuild
This reverts commit 8f21dc5.
Description
gatsby-plugin-image
)gatsby-plugin-image
public/functions
for compatibility between Gatsby Cloud builds and Netlify servingdev
branch (includingdev
)dev
intomaster
, which will build final production build frommaster
branch.master
branch using existing Netlify account.ethereum.org
Related Issue
#4271