-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
Cannot find module '../native' (when using V3 spec beta release) #514
Comments
@mefellows @uglyog : I encountered this too, and fixed it by modifying the import to |
It may do. My recollection of the issue is that if pact is installed at a different level (e.g. within another dependency or lerna hierarchy) the relative path isn't resolved correctly. We'll need to reliably reproduce it to fix first. |
I think the problem is more to do with node module resolution - index.node isn’t always picked up as a candidate for index.
My suspicion is that this might only be a problem for typescript users, because the documentation says that index includes work “the same as node”, but only mention index.js, not index.node
…Sent from my mobile
On 7 Oct 2020, at 9:36 am, Matt Fellows ***@***.***> wrote:
It may do. My recollection of the issue is that if pact is installed at a different level (e.g. within another dependency or lerna hierarchy) the relative path isn't resolved correctly.
We'll need to reliably reproduce it to fix first.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I've updated the imports, but I haven't been able to replicate this issue. It was worked ok for my test projects. |
Ah! due to the |
This is probably the case cause we are using yarn workspaceses and |
@aelmekeev Does v10.0.0-beta.17 work for you? |
@TimothyJones it does work 🎉 |
Great! It works for jest-pact's beta V3 build too, so I think we can close this. |
Copied from another issue about V3:
I'm having similar issue with 10.x as well. But we are not using
jest-pact
and the error isOriginally posted by @aelmekeev in #483 (comment)
The text was updated successfully, but these errors were encountered: