Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

cherry-pick(#32491): fix(test runner): allow directory imports with path mapping #32571

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

dgozman
Copy link
Contributor

@dgozman dgozman commented Sep 11, 2024

We now hopefully align with moduleResolution: bundler tsconfig option, allowing directory imports in every scenario, and allowing proper module imports when not going through the type mapping.

This regressed in #32078. Fixes #32480, fixes #31811.

…ts with path mapping

We now hopefully align with `moduleResolution: bundler` tsconfig option,
allowing directory imports in every scenario, and allowing proper module
imports when not going through the type mapping.

This regressed in microsoft#32078. Fixes microsoft#32480, fixes microsoft#31811.
Copy link
Contributor

Test results for "tests 1"

1 failed
❌ [playwright-test] › babel.spec.ts:135:5 › should not transform external

1 flaky ⚠️ [playwright-test] › ui-mode-test-setup.spec.ts:22:5 › should run global setup and teardown

29575 passed, 629 skipped
✔️✔️✔️

Merge workflow run.

@dgozman dgozman merged commit 13e6e48 into microsoft:release-1.47 Sep 12, 2024
29 of 30 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant