From 0a15b5f8cc035f033d5ef2465672295810934e30 Mon Sep 17 00:00:00 2001 From: Lukas Taegert-Atkinson Date: Sat, 28 May 2022 15:01:42 +0200 Subject: [PATCH] Update link to node polyfill repo (#4513) --- cli/run/batchWarnings.ts | 2 +- docs/08-troubleshooting.md | 2 +- src/finalisers/shared/warnOnBuiltins.ts | 2 +- test/cli/samples/warn-multiple/_config.js | 2 +- test/misc/misc.js | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/cli/run/batchWarnings.ts b/cli/run/batchWarnings.ts index 1180a517c72..2bc57688509 100644 --- a/cli/run/batchWarnings.ts +++ b/cli/run/batchWarnings.ts @@ -78,7 +78,7 @@ const immediateHandlers: { stderr( `Creating a browser bundle that depends on ${printQuotedStringList( warning.modules! - )}. You might need to include https://github.com/snowpackjs/rollup-plugin-polyfill-node` + )}. You might need to include https://github.com/FredKSchott/rollup-plugin-polyfill-node` ); }, diff --git a/docs/08-troubleshooting.md b/docs/08-troubleshooting.md index 753faab44f8..7ed5f3e30c1 100644 --- a/docs/08-troubleshooting.md +++ b/docs/08-troubleshooting.md @@ -86,7 +86,7 @@ export default { If you _do_ want to include the module in your bundle, you need to tell Rollup how to find it. In most cases, this is a question of using [@rollup/plugin-node-resolve](https://github.com/rollup/plugins/tree/master/packages/node-resolve). -Some modules, like `events` or `util`, are built in to Node.js. If you want to include those (for example, so that your bundle runs in the browser), you may need to include [rollup-plugin-polyfill-node](https://github.com/snowpackjs/rollup-plugin-polyfill-node). +Some modules, like `events` or `util`, are built in to Node.js. If you want to include those (for example, so that your bundle runs in the browser), you may need to include [rollup-plugin-polyfill-node](https://github.com/FredKSchott/rollup-plugin-polyfill-node). ### Error: "EMFILE: too many open files" diff --git a/src/finalisers/shared/warnOnBuiltins.ts b/src/finalisers/shared/warnOnBuiltins.ts index 59d0f0cf961..6457adf1127 100644 --- a/src/finalisers/shared/warnOnBuiltins.ts +++ b/src/finalisers/shared/warnOnBuiltins.ts @@ -38,7 +38,7 @@ export default function warnOnBuiltins( code: 'MISSING_NODE_BUILTINS', message: `Creating a browser bundle that depends on Node.js built-in modules (${printQuotedStringList( externalBuiltins - )}). You might need to include https://github.com/snowpackjs/rollup-plugin-polyfill-node`, + )}). You might need to include https://github.com/FredKSchott/rollup-plugin-polyfill-node`, modules: externalBuiltins }); } diff --git a/test/cli/samples/warn-multiple/_config.js b/test/cli/samples/warn-multiple/_config.js index 351dd8dd998..45e8770b975 100644 --- a/test/cli/samples/warn-multiple/_config.js +++ b/test/cli/samples/warn-multiple/_config.js @@ -7,7 +7,7 @@ module.exports = { assertIncludes( stderr, '(!) Missing shims for Node.js built-ins\n' + - 'Creating a browser bundle that depends on "url", "assert" and "path". You might need to include https://github.com/snowpackjs/rollup-plugin-polyfill-node\n' + 'Creating a browser bundle that depends on "url", "assert" and "path". You might need to include https://github.com/FredKSchott/rollup-plugin-polyfill-node\n' ); assertIncludes( stderr, diff --git a/test/misc/misc.js b/test/misc/misc.js index 053c8e5c875..76132722719 100644 --- a/test/misc/misc.js +++ b/test/misc/misc.js @@ -49,7 +49,7 @@ describe('misc', () => { assert.equal(relevantWarnings.length, 1); assert.equal( relevantWarnings[0].message, - `Creating a browser bundle that depends on Node.js built-in modules ("util"). You might need to include https://github.com/snowpackjs/rollup-plugin-polyfill-node` + `Creating a browser bundle that depends on Node.js built-in modules ("util"). You might need to include https://github.com/FredKSchott/rollup-plugin-polyfill-node` ); }); });