From d6cebc2466036e4f9bfdfd33edb82c721ab77e59 Mon Sep 17 00:00:00 2001 From: Will Binns-Smith Date: Mon, 23 Sep 2019 21:45:06 +0000 Subject: [PATCH] Merged in wbinnssmith/add-back-packaging-invariant (pull request #1) Restore invariant asserting bundles don't have external assets Approved-by: Trey Shugart --- packages/packagers/js/src/JSPackager.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/packagers/js/src/JSPackager.js b/packages/packagers/js/src/JSPackager.js index 336f09169e44..995c22bdba4a 100644 --- a/packages/packagers/js/src/JSPackager.js +++ b/packages/packagers/js/src/JSPackager.js @@ -54,12 +54,12 @@ export default new Packager({ ) { // if this is a reference to another javascript asset, we should not include // its output, as its contents should already be loaded. - // invariant( - // !bundle.hasAsset(resolved), - // `Expected JS bundle ${ - // bundle.name - // } to only contain JS, but contained ${resolved.filePath}` - // ); + invariant( + !bundle.hasAsset(resolved), + `Expected JS bundle ${ + bundle.name + } to only contain JS, but contained ${resolved.filePath}` + ); wrapped += JSON.stringify(resolved.id) + ':[function() {},{}]'; } else { return;