fix: benchmarks #1542
fix: benchmarks #1542
Conversation
|
|
I dont think this needs a changelog since we dont require a package bump. Happy to open one if needed though |
| const code = fs | ||
| .readFileSync(path.resolve(__dirname, '../../dist/rrweb.umd.cjs')) | ||
| .toString(); | ||
| await page.setContent(`<html> | ||
| <script> | ||
| ${code.toString()} | ||
| </script> | ||
| </html>`); |
There was a problem hiding this comment.
What is the reason for moving this (back) to an inlined script element? This should make the profiles harder to read as the source map references are no longer intact.
Fixes the benchmark runs and adds a title so its easier to tell what suite was profiled and make sense of the profiles