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

Fixes directories masking files with the same name #6

Closed
wants to merge 1 commit into from

Conversation

kherock
Copy link

@kherock kherock commented May 5, 2020

This attempts to fix #4 by attempting to match files without the trailing slash before falling back to files beneath a matching directory.

I've created a repo ts-pnp-bugs reproducing these bugs with a couple major libraries (aws-sdk and lodash/fp), and tested these changes against that repo. I have a medium-sized workspace I'm attempting to migrate to Yarn 2 + PnP and these changes unblock me.

@gausie
Copy link

gausie commented May 4, 2021

I did this by adding a !moduleResolutionHost.fileExists(unqualified) to the if

@kherock kherock closed this Oct 29, 2021
@kherock kherock deleted the fix-directory-masking branch November 13, 2021 02:26
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.

module resolution logic doesn't match tsc
2 participants