Greg Bergé

Results 229 comments of Greg Bergé

I prefer to keep it explicit as it is in the current version.

Hi @gvanhorne, thanks for submitting the issue. If someone could help fix this it's welcome, here is a Stackoverflow post about the issue: https://stackoverflow.com/questions/57572062/puppeteer-web-protocol-error-target-getbrowsercontexts-not-allowed

Hello @jamesgpearce, maybe you should fill an issue in Puppeteer. Because browser should be ready, I mean there is no reason to warm it up on our side.

I will not accept it. "main" is not part of the spec for ESM module. Post an issue in Metro bundler instead. This package is correct, the bundler is not.

Gently bumping this one. @colincasey any news on that?

> Should we merge or delete this PR? Not yet done. I will close it for now.

> related #678 I mitigated this bug by downgrading Sentry, but even by doing so, the instrumentation file is not called at all. So it's two different problemd. #678 is...