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: bump minimum node version to v18 #196

Closed
wants to merge 2 commits into from

Conversation

browniebroke
Copy link

@browniebroke browniebroke commented Nov 11, 2022

I was trying to upgrade my site to Gatsby v5 and noticed this plugin was blocking me. Sadly I cannot confirm whether this work on my site since I'm also blocked by another plugin, but sharing in case other can test/pick up the work from here.

EDIT: other plugin has been updated, my site deploy seems to work and the headers are correctly picked up

EDIT 2: This was initially about upgrading to Gatsby v5 but that part was superseded in #198. However, Gatsby v5 only supports node 18, so the minimum node version for this plugin can also be bumped.

Relates #195

@browniebroke browniebroke requested a review from a team November 11, 2022 17:29
@browniebroke browniebroke changed the title feat: add support for Gatsby v5 fix: bump minimum node version to v18 Nov 28, 2022
@ericapisani
Copy link
Contributor

Hi @browniebroke ,

Thanks for opening up a pull request! We've made the changes needed to in order to support Gatsby 5 within this plugin (released as part of 5.1.0, so I'm going to close this since this is no longer needed.

Assuming you also need the related gatsby v5 support changes in the @netlify/plugin-gatsby, those will be released as part of this PR

@browniebroke browniebroke deleted the gatsby-v5 branch November 30, 2022 20:55
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