Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Similar to #11359 and #11360, I had problems when using jest-diff together with rollup's plugin-commonjs that turns cjs back into esm. It appears that there was originally a desire to use all named exports for jest-diff, but the move was delayed originally. This PR removes the default export for
jest-diff
and exposesdiff()
as a named export. I also did my best to find any usages of the default export in the documentation.Test plan
I tested this in my own snowpack app by building jest from my branch and replacing files in my
node_modules
.