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

[DNM][experimental][WIP Yarn workspaces to help with Lerna #5665

Closed
wants to merge 10 commits into from
Closed

[DNM][experimental][WIP Yarn workspaces to help with Lerna #5665

wants to merge 10 commits into from

Conversation

acourdavault
Copy link
Member

@acourdavault acourdavault commented Oct 29, 2019

Description

with Lerna some yarn command do not work
image

I read this:
https://yarnpkg.com/blog/2017/08/02/introducing-workspaces/
https://doppelmutzi.github.io/monorepo-lerna-yarn-workspaces/

And tried to see if enabling Yarn workspace would help

Type of Change

  • Non breaking change ('update', 'change')

change

Added this to /package.json

  "workspaces":[
    "packages/*"
  ]

Yarn should not have to be configures as workspaces is enabled by default

After doing this, and rerunning yarn install from the root everything works.

It is compatible with lerna

After this i can run in manager and root and linode-js-sdk:

  • audit
  • outdated
  • upgrade

BUT

yarn install in packages/manager/ Still fails because of patch-packages.
Although supposedly it should support it :
ds300/patch-package#42

Note to Reviewers

DO NOT MERGE, we need to talk and see if we think this is useful

@acourdavault
Copy link
Member Author

Made a cleaner version of this

@acourdavault acourdavault deleted the fix-yarn-cmd-fail-lerna branch October 30, 2019 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants