Conversation
✅ Deploy Preview for babel-next ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for babel ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| nodeLinker: node-modules | ||
|
|
||
| yarnPath: .yarn/releases/yarn-4.6.0.cjs | ||
| yarnPath: .yarn/releases/yarn-4.12.0.cjs |
There was a problem hiding this comment.
Would you mind if I opened a PR and replaced it with npm or pnpm?
yarnpkg/berry#6999
There was a problem hiding this comment.
We already use yarn in the main repo. If you are blocked by that yarn issue, we can patch the yarn used in Babel repos.
There was a problem hiding this comment.
Thanks! It's just a bit inconvenient, but I can work around this issue for now.
Let's talk about that when we want to upgrade to Yarn 6.
There was a problem hiding this comment.
Oh OK, I went ahead and patched the yarn version. Do you want to revert the patch or keep it?
There was a problem hiding this comment.
Let's keep it in this repository, since the default preview command requires this patch. Other repositories rarely use yarn run.
There was a problem hiding this comment.
Yeah it is trivial to apply the patch, just search and replace @goto: If you prefer you can also open a similar PR to the main repo. As I remember it's not the first time we patch yarn. For the time being I will keep this patch as it may benefit other potential contributors using Windows.
Also removed
lodash.camelcase.