Skip to content
This repository was archived by the owner on Apr 19, 2021. It is now read-only.

Commit

Permalink
šŸ› fix: workaround Sapper's warning-swallowing issue until it's solvedā€¦
Browse files Browse the repository at this point in the history
ā€¦ upstream
  • Loading branch information
babichjacob committed May 28, 2020
1 parent 719b484 commit de86997
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,8 @@ const warningIsIgnored = (warning) => warning.message.includes(
"Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification",
) || warning.message.includes("Circular dependency: node_modules");

const onwarn = (warning, onwarn_) => (warning.code === "CIRCULAR_DEPENDENCY" && /[/\\]@sapper[/\\]/.test(warning.message)) || warningIsIgnored(warning) || onwarn_(warning);
// Workaround for https://github.com/sveltejs/sapper/issues/1221
const onwarn = (warning, _onwarn) => (warning.code === "CIRCULAR_DEPENDENCY" && /[/\\]@sapper[/\\]/.test(warning.message)) || warningIsIgnored(warning) || console.warn(warning.toString());

export default {
client: {
Expand Down

0 comments on commit de86997

Please sign in to comment.