Yagiz Nizipli

Results 171 issues of Yagiz Nizipli

### Description of the bug: - Clone github.com/cloudflare/workerd - Open clion as bazel project - Use BUILD.bazel - Sync - Go to `src/workerd/api/node/buffer.h` ![image](https://github.com/user-attachments/assets/6befdfcc-177a-4ae8-a4e3-432450ef54b6) The recommendation is correct. The file...

type: bug
product: IntelliJ
awaiting-maintainer

Updates all eslint dependencies to latest versions, and update inconsistencies with prettier, typescript-eslint and eslint recommended configurations. Fixes https://github.com/cloudflare/workerd/issues/2551

needs-internal-pr

We currently roll our own implementation for convenience methods that doesn't support `{info:true}` options. Node.js implementation which lives under https://github.com/nodejs/node/blob/main/lib/zlib.js#L160 is as follows: ```js function zlibBufferOnEnd() { let buf; if...

bug

We are using 8.57.0 as of August 19. The latest version is v9. We eventually need to update to latest version of Eslint to update our dependencies as well. -...

Following up for the changes on https://github.com/web-platform-tests/wpt/pull/48084, I'd like to propose adding 2 Bazel files to web-platform tests to make integrations as seamless as possible. cc @jasnell Note: This RFC...

We have been using Ada's ToASCII and ToUnicode implementations since Node 18. I think it's safe to assume we can remove this "unused" code.

c++
i18n-api
needs-ci

An attempt to modernize C++ code to use `[[likely]]` and `[[unlikely]]` available after C++20. Reference: https://en.cppreference.com/w/cpp/language/attributes/likely This pull-request also disables `readability/braces` rule from cpplint, since it doesn't support C++20. Main...

c++
lib / src
needs-ci
commit-queue
dont-land-on-v18.x
dont-land-on-v20.x

### Test `test-async-context-frame` ### Platform Windows ### Console output ```console duration_ms: 6718.227 exitcode: 1 severity: fail stack: "\u25B6 AsyncContextFrame\n \u2714 async-hooks\\test-async-local-storage-args.js\ \ (4955.4808ms)\n \u2714 async-hooks\\test-async-local-storage-async-await.js (4400.0139ms)\n\ \ \u2714 async-hooks\\test-async-local-storage-async-functions.js (5106.203ms)\n\...

windows
flaky-test

What if we just called the shell directly to avoid the cost of booting a Node.js instance? cc @nodejs/build any concerns?

test
needs-ci
review wanted

Node.js v22+ supports `Promise.withResolvers()`. Let's use them!

lib / src
promises
blocked
needs-ci
needs-benchmark-ci
dont-land-on-v18.x
dont-land-on-v20.x