-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
[jest-haste-map] Remove mapper
option.
#9581
Conversation
1039c42
to
550d6c1
Compare
cc @arcanis |
Thanks for the ping - Yarn doesn't rely on this option, lgtm 👍 |
Codecov Report
@@ Coverage Diff @@
## master #9581 +/- ##
==========================================
+ Coverage 65.07% 65.08% +<.01%
==========================================
Files 286 286
Lines 12144 12137 -7
Branches 3010 3007 -3
==========================================
- Hits 7903 7899 -4
+ Misses 3605 3604 -1
+ Partials 636 634 -2
Continue to review full report at Codecov.
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
This option was added a while back to support some use-cases with Yarn PnP but it was neither documented nor is this how I would solve the problem of files not being materialized as this functionality has the potential to de-opt operations in Jest's haste map.
This PR removes the option.
Test plan
yolo