Skip to content

fix: prevent async imports from creating chunks#704

Merged
zamotany merged 1 commit into
callstack:fix/use-signle-chunkfrom
hedgepigdaniel:fix/avoid-multiple-chunks
Mar 2, 2020
Merged

fix: prevent async imports from creating chunks#704
zamotany merged 1 commit into
callstack:fix/use-signle-chunkfrom
hedgepigdaniel:fix/avoid-multiple-chunks

Conversation

@hedgepigdaniel
Copy link
Copy Markdown
Contributor

Summary

Fixes #701

Test plan

@zamotany
Copy link
Copy Markdown
Contributor

zamotany commented Feb 21, 2020

Thanks for the PR, looks good, but I need to check the failing tests. I suspect it's because of assertion that doesn't make sense now. Basically it was checking if the following regex is matched in the bundle:

/return asyncEval\(__webpack_require__\.p \+ "" \+ chunkId \+ "\.app1\.(ios|android)\.bundle"\);/g

but if the modules from chunk are inside the bundle, this code will never be created. Also we can probably remove code that add asyncEval helper functions.

@zamotany zamotany changed the base branch from master to fix/use-signle-chunk March 2, 2020 14:51
@zamotany zamotany merged commit 05ac921 into callstack:fix/use-signle-chunk Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

--sourcemap-output + async import() = Multiple assets emit different content to the same filename

2 participants