feat: add buildDefaultResolverRequire#9194
feat: add buildDefaultResolverRequire#9194SimenB merged 2 commits intojestjs:masterfrom G-Rath:implement-buildDefaultResolverRequire-function
buildDefaultResolverRequire#9194Conversation
|
Once merged this can replace the following: i.e /cc @M4rk9696 |
Codecov Report
@@ Coverage Diff @@
## master #9194 +/- ##
==========================================
- Coverage 65.1% 65.06% -0.05%
==========================================
Files 278 278
Lines 11860 11865 +5
Branches 2923 2925 +2
==========================================
- Hits 7722 7720 -2
- Misses 3511 3517 +6
- Partials 627 628 +1
Continue to review full report at Codecov.
|
SimenB
left a comment
SimenB
left a comment
There was a problem hiding this comment.
should be exported from packages/jest-transform/src/index.ts. Also needs a changelog entry. It adds a public function, so I think it deserves its own mention
SimenB
changed the title
buildDefaultResolverRequire functionbuildDefaultResolverRequire
|
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. |
4 participants
This is a breaking out of
buildDefaultResolverRequirefrom #8829 as requested@SimenB I've not written tests or add an entry in the changelog as it seemed like a bit of overkill for something that is going to be used in future PRs.
I did a sweep of the codebase, and the only place this could actually be used right now is in
jest-core, but that'd require implementing support for the callback parameters 🤷♀It'll be nice to trim down the transformer PRs a bit :)