Skip to content

Commit

Permalink
Suppress warning about eval usage by the syn dependency
Browse files Browse the repository at this point in the history
From a quick glance it looks like the code exists to handle `javascript:` anchor
links, which we don't use.
  • Loading branch information
robertknight committed Nov 25, 2024
1 parent 31dd3dd commit b7269e2
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions rollup-tests.config.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { log } from '@hypothesis/frontend-build';
import { babel } from '@rollup/plugin-babel';
import commonjs from '@rollup/plugin-commonjs';
import { nodeResolve } from '@rollup/plugin-node-resolve';
Expand Down Expand Up @@ -57,4 +58,12 @@ export default {
],
}),
],
onwarn: warning => {
// Suppress security warning about `eval` usage in syn, a test-only
// dependency.
if (warning.id.includes('node_modules/syn/') && warning.code === 'EVAL') {
return;
}
log.warn(`Rollup warning: ${warning} (${warning.url})`);
},
};

0 comments on commit b7269e2

Please sign in to comment.