-
-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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: require.resolve to correct sub node_modules #3003
Conversation
Excuse me @Shinigami92 when the PR can be checked and merged? My framework which used by vite need the bugfix |
Thanks for the PR @zhangyuang, LGTM. Others in the team need to review it to merge it now. There are a lot of PRs and issues due to the explosion in usage of Vite. To avoid being unlocked you can use something like https://www.npmjs.com/package/patch-package to patch your Vite version until this fix is released. |
@zhangyuang Yeah, I'm sorry I'm currently unable to review your PR now Hopefully antfu or I can get into a review the next few days |
Description
for the issue
when the same module for example
path-to-regexp
has different version in repo, prebundle will be error.Additional context
What is the purpose of this pull request?