-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
chore(project): update to node 14.17 lts #8704
Conversation
Deploy Preview for carbon-elements ready! Built with commit 437ff1f |
Deploy Preview for carbon-components-react ready! Built without sensitive environment variables with commit 437ff1f https://deploy-preview-8704--carbon-components-react.netlify.app |
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.
Looks great! I think all that's left is running yarn dedupe
and you'll be good to go 👍
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.
Everything seems to be working fine on my end 👍🏻 ✅
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.
@sstrubberg should we also update the configs for circle ci and the github action workflows to be explicitly 14.17 vs 14.x?
@tay1orjones in the other PR we were talking about it and 14.x should resolve to 14.17 now that it's LTS so it should be good 👀 |
This updates node to v14.17 LTS.
Changelog
Changed
node-sass
&gulp-sass
dependencies across the project to allow build process to complete successfully.yarn dedupe
to remove dupes.Testing / Reviewing
Make sure this doesn't break our build!