-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
chore: update some dependencies #2671
Conversation
✅ Deploy Preview for babel ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This seems to require upgrading the yarn1 version in the netlify environment variable settings. Can you help me? |
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.
Does it work if you keep .yarnrc
? I guess older Yarn 1 versions, like 1.13 used in netlify build images, only supports .yarnrc
.
It doesn't seem possible, due to the old version of node. https://answers.netlify.com/t/default-yarn-version-is-now-1-17/2297/11 |
@liuxingbaoyu I have added |
This reverts commit 160e944.
It looks like we also need to update the NODE version as yarn3 dropped NODE10 support. |
https://docs.netlify.com/configure-builds/manage-dependencies/#node-js-and-javascript Can you create |
Yarn is working fine now, there are other errors, I will continue to investigate. |
https://docs.netlify.com/configure-builds/troubleshooting-tips/#build-cache |
@liuxingbaoyu The test was passing (but failing on CI) probably because our local filesystems are not case-sensitive. |
Amazing, thanks! |
webpack doesn't work on newer versions of nodejs.