Skip to content
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: don't try to resolve internal node modules via package.json svelte field #266

Merged
merged 2 commits into from
Feb 4, 2022

Conversation

dominikg
Copy link
Member

@dominikg dominikg commented Feb 3, 2022

i could swear is saw a response in a closed issue that node internals were printed as missing a package.json export in buildEnd.
Can't find that now but we shouldn't even be trying, even if vite is asking our resolve hook.

found it: #244 (comment)

@dominikg dominikg merged commit ed8a19c into main Feb 4, 2022
@dominikg dominikg deleted the fix/ignore-internal-modules branch February 4, 2022 09:09
@github-actions github-actions bot mentioned this pull request Feb 4, 2022
@github-actions github-actions bot mentioned this pull request Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants