-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
packages are linked incorrectly on windows #4135
Comments
Is it something we do wrong, or maybe a Lerna bug? |
cc @bestander |
Will look at it today
…On Thu, Jul 27, 2017 at 7:22 AM Christoph Nakazawa ***@***.***> wrote:
cc @bestander <https://github.com/bestander>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4135 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ACBdWBci4FBih5922ZK5t8iS5mdjFvN_ks5sSJ0YgaJpZM4Okliw>
.
|
@aaronabramov I don't have a windows PC at this moment. What does https://ci.appveyor.com/project/Daniel15/jest/build/4233 line 80-125 tell you? |
@bestander seems like yarn update fixed the issue. line 129 prints out the path to the local package directory (unlike mine PR where it goes to |
Reopening, we have two PRs hitting that |
This belongs on the Yarn issue tracker. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
here's a repro build: https://ci.appveyor.com/project/Daniel15/jest/build/4207
PR: #4134
In the logs you can see that
require.resolve('jest-config')
returns:\projects\jest\node_modules\jest-config\build\index.js
which seems to be the last published version of the package from npm.locally and on circle/travis it resolves to
jest/packages/jest-config/build/index.js
, the local version of the file.The text was updated successfully, but these errors were encountered: