10000 Try file if matched pattern specifies the extension instead of all the time by sheetalkamat · Pull Request #42246 · microsoft/TypeScript · GitHub
[go: up one dir, main page]

Skip to content

Try file if matched pattern specifies the extension instead of all the time #42246

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

Merged
merged 2 commits into from
Jan 8, 2021

Conversation

sheetalkamat
Copy link
Member

Instead of looking for extension on resulting path, look if substitution of path mapping specifies the extension. Then only try the file as is, otherwise do normal module resolution for the resulting path
Fixes #39743

@typescript-bot typescript-bot added Author: Team For Milestone Bug PRs that fix a bug with a specific milestone labels Jan 7, 2021
@sheetalkamat sheetalkamat marked this pull request as ready for review January 7, 2021 22:32
@sheetalkamat
Copy link
Member Author

@typescript-bot test this
@typescript-bot user test this

@typescript-bot
Copy link
Collaborator
typescript-bot commented Jan 7, 2021

Heya @sheetalkamat, I've started to run the parallelized community code test suite on this PR at cfe6b69. You can monitor the build here.

@typescript-bot
Copy link
Collaborator
typescript-bot commented Jan 7, 2021

Heya @sheetalkamat, I've started to run the extended test suite on this PR at cfe6b69. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master.

@sheetalkamat sheetalkamat merged commit d36df0d into master Jan 8, 2021
@sheetalkamat sheetalkamat deleted the patternMatchingFix branch January 8, 2021 20:03
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
…e time (microsoft#42246)

* Test case for path mapping with extension

* Try file if matched pattern specifies the extension instead of all the time
Fixes microsoft#39743
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Milestone Bug PRs that fix a bug with a specific milestone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't find declaration files for imports with extension when using path mapping
3 participants
0