-
-
Notifications
You must be signed in to change notification settings - Fork 431
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
Fix dependency resolution when using pnpm #774
Conversation
I attempted to create a
After So in order to create a unit test I would need to include a real npm package that exhibits this re-export behavior, one that I know of is |
@xbtsw the way pnpm currently works, it only symlinks directory dependencies but does not run install in the external package. Though you can run |
It turns out to be using a folder dependency cannot trigger the error. The dependencies needs to be downloaded from a remote repository. |
I am okay with that! Go forth and conquer! |
@johnnyreilly The test is added. I am ready to merge. Please kindly review and comment 🤝 |
Will do! I'm away from a keyboard for the next 5 days or so. Will check when I'm back. Thanks for your work! |
Thanks for your work! Released with https://github.com/TypeStrong/ts-loader/releases/tag/v4.3.0 |
realpath
tomoduleResolutionHost
so that TypeScript could follow symlink as it supposed to beFix #768