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

Support for Node 17 #3194

Closed
6 tasks done
nschonni opened this issue Oct 15, 2021 · 9 comments
Closed
6 tasks done

Support for Node 17 #3194

nschonni opened this issue Oct 15, 2021 · 9 comments

Comments

@nschonni
Copy link
Contributor

nschonni commented Oct 15, 2021

This new major version is supposed to be released on October 19th. Opening this as a placeholder for when we start to get "node-sass won't install" issues.

/cc @saper @xzyfer

@nschonni
Copy link
Contributor Author

@xzyfer not sure if we want to try and add #3193 or just the setUnauthorized one

@quisido
Copy link

quisido commented Nov 13, 2021

I know this says Node 17 released October 19th, but it wasn't until sometime between Nov 4-10 that GitHub Actions upgraded their Node version for lts/* to 17.x, causing all node-sass dependencies to fail to install.

I'm not sure if this change by GitHub Actions impacts the timelines or priorities of this ticket at all.

EDIT: @nschonni is correct. The change to GitHub Actions was to now use 16.x as lts/* instead of 14.x. The failure for node-sass@5.0.0 is likely expected. React projects should yarn remove node-sass && yarn add --dev sass to resolve the issue.

@nschonni
Copy link
Contributor Author

Node version for lts/* to 17.x,

No, 17 is not LTS, that is was bumped from 14 to 16 around that date. 17 will never be a LTS release. If that is happening that would be a bug with nvm/GitHub Actions

The work for 17 was mostly done in the other PR, but binaries still need to be cut and an NPM release made.

@xzyfer
Copy link
Contributor

xzyfer commented Nov 28, 2021

Apologies for the delay I've been job hunting and moving house.

Thanks for this @nschonni. I've merged #3149 and will try to cut a release in the next couple hours.

@xzyfer
Copy link
Contributor

xzyfer commented Nov 28, 2021

I ran into some issues with the build scripts. Managed to get the windows, osx, linux, and alpine Node 17 binaries uploaded to the v7.0.0 prerelease before going to bed. Will need to copy over the remaining binaries from the v6.0.1 release tomorrow.

@RochelleLong

This comment has been minimized.

@xzyfer
Copy link
Contributor

xzyfer commented Dec 6, 2021

I ran into some issues with my release script but it's resolved now. V7 has been released with Node 17 support.

@xzyfer xzyfer closed this as completed Dec 6, 2021
@RochelleLong

This comment has been minimized.

@xzyfer
Copy link
Contributor

xzyfer commented Dec 6, 2021 via email

@xzyfer xzyfer unpinned this issue Dec 28, 2021
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

No branches or pull requests

4 participants