-
Notifications
You must be signed in to change notification settings - Fork 13
Handle new yarn 4 requirement #17
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
Conversation
|
This version still throws a |
|
Ah, you have to do a I guess we need to add that to the add-on, or else do this a different way. Thanks @T-Fletcher |
|
Thanks for the careful look @T-Fletcher - Please try again with Also, please use the updated steps in your https://github.com/justafish/ddev-drupal-core-dev/pull/16/files |
|
Thanks @rfay, I can confirm your latest version works for me with the updated steps |
|
Thank you! |
There are a couple of problems with the new yarn v4 requirements for core.
This PR
ddev yarnin thecoredirectory, which seems to work whereas theddev exec yarn --cwddoes not, which I assume is a bug in yarn/nodeTest with