-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support lerna projects with yarn workspace integration #5
Comments
Hmm I am not testing with yarn or lerna, but feel free to add this feature and submit a pull request. Look how it copies files - you just need to somehow pass the correct folder cypress.
…Sent from my iPhone
On Jan 28, 2018, at 09:46, Jaime Sangcap ***@***.***> wrote:
Feature request
I'm using Lerna with yarn workspace integration. Below is my folder structure
* F:\greenfield\taskify
| ----packages/
| --------client/
| ------------package.json
| --------server/
| ------------package.json
| --------e2e/
| ------------cypress/
| ------------package.json
| ----package.json
| ----lerna.json
When I run the command (actual command after $):
F:\greenfield\taskify$ lerna add --scope=e2e @bahmutov/add-typescript-to-cypress --dev
or
F:\greenfield\taskify\packages\e2e $ yarn add --dev @bahmutov/add-typescript-to-cypress
I'm having the below error.
error ***@***.***\add-t
ypescript-to-cypress: Command failed.
Exit code: 1
Command: node src/add-plugin.js
Arguments:
Directory: ***@***.***\
add-typescript-to-cypress
Output:
|
Thank you for super quick response 😄 I'm just new to js/nodejs/npm world but I'm very willing to take a stab on this one. |
Yeah, look at this file https://github.com/bahmutov/add-typescript-to-cypress/blob/master/src/add-plugin.js that is executed |
the postinstall works now, I've used lerna without yarn workspace integration. I will send a PR for the docs |
@jaimesangcap You can use For example, in a package.json:
|
I ran across this Issue today. I use lerna without yarn workspaces and therefore can't use nohoist in my package.json I made it work with But I would still prefer to have this fixed within this package. But I am not sure how tell it prior to installing it where the cypress folder is actually located. Any ideas how such a fix could look like? |
Feature request
I'm using Lerna with yarn workspace integration. Below is my folder structure
When I run the command (actual command after $):
F:\greenfield\taskify$ lerna add --scope=e2e @bahmutov/add-typescript-to-cypress --dev
or
F:\greenfield\taskify\packages\e2e $ yarn add --dev @bahmutov/add-typescript-to-cypress
I'm having the below error.
It seems like the
process.cwd()
is picking up the project root directory every time.I've tried googling if it's possible to set the
cwd
when running the command but no luck.The text was updated successfully, but these errors were encountered: