190n

Results 44 comments of 190n

I realize we never got test coverage for this change on macOS because we don't enable assertions there.

> Maybe instead, rewatchPosix in the success case should this.ref()? That sounds good, we'll see if it works

This is related to V8 API support. The reason you see that issue is that you have an older version of Node.js installed, which is different than the version that...

This sounds good but I'm also concerned about the name. They have limited precision and limited range, unlike real numbers (obviously those limits can't be overcome in a fixed-size type,...

I've made a PR at oven-sh/bun#18172 that may help with the `error: Unexpected` issue, but I'm not certain since I can't repro the error with ConvertX on my own machine....

That PR is merged now so you can also try using the `canary-alpine` tag on our Docker image, instead of manually replacing the Bun binary.

Oh, you were able to test and see that canary fixed the issue?

I reported a different, but similar-looking, crash to WebKit at https://bugs.webkit.org/show_bug.cgi?id=289009. I'm hoping that maybe the fix for that will also benefit pglite. @sroussey, normally I would suggest you turn...

Can you share the script you're running that has this issue?

The error this code produces in other runtimes is: ``` error: Uncaught SyntaxError: Invalid left-hand side in assignment for ([{ ^ ``` Bun transpiles all code before running it. In...