-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
issue with windows build lerna #3427
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
base: dev
Are you sure you want to change the base?
Conversation
|
@T4rk1n this looks good to me - please merge when you're next doing a patch |
|
@T4rk1n could you take a look and see if this is still good to merge? |
| ) | ||
|
|
||
| cmdstr = f"npx lerna exec --concurrency {concurrency} --scope='{source_glob}' -- npm {install_type}" | ||
| cmdstr = f"npx lerna exec --concurrency {concurrency} --scope={source_glob} -- npm {install_type}" |
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.
We needed those '' for spaces that might be in the directory path, while ' is not windows compatible, " is and work on both os.
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.
Just tried it, and it did not work.
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.
It was an issue with the posix, needed to be passed True
With the upgrade of lerna, windows ceased to build: