Merge pull request #1552 from hypothesis/fix-external-require-reference

Fix module load error on pages defining `require`

Merge pull request #1552 from hypothesis/fix-external-require-reference
Fix module load error on pages defining `require`
96389f20 · Robert Knight · GitHub · 1eff5f68 · 3f9bc92d · 96389f20
Unverified Commit 96389f20 authored Dec 02, 2019 by Robert Knight Committed by GitHub Dec 02, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

create-bundle.js scripts/gulp/create-bundle.js +2 -2

No files found.
--- a/scripts/gulp/create-bundle.js
+++ b/scripts/gulp/create-bundle.js
@@ -135,8 +135,8 @@ module.exports = function createBundle(config, buildOpts) {
    .readFileSync(defaultPreludePath)
    .toString()
    .replace(
-      'var previousRequire = typeof require == "function" && require;',
-      `var previousRequire = typeof ${externalRequireName} == "function" && ${externalRequireName};`
+      /typeof require == "function" && require/g,
+      `typeof ${externalRequireName} == "function" && ${externalRequireName};`
    );
  if (!prelude.includes(externalRequireName)) {
    throw new Error(