Kerem Kat

Results 36 comments of Kerem Kat

I have added javascript tests to the new file `explain_invalid_arguments.js`.

Thank you for the explanations. It seems that tests are using the explain command in an undocumented way, at least I could not find any documentation for this behavior: ```js...

Rebased and formatted with `clang-format-7.0.1`.

Hi @dstorch, Thank you for the detailed and helpful review. I am interested in making the necessary changes to this PR after the rewriting of the implementation of projection execution...

Merge conflicts are resolved now, without resolving any of your comments.

Hi @dstorch, I have removed the `ShardNameStage` and moved the functionality to the `ShardFilterStage`. Now the `ShardingFilterNode` constructor has a `wantShardName` parameter - not sure if this is allowed. The...

Maybe this year this PR will get some love :) @dstorch

Hi! Will there be a release between now and the LTS version? Would the LTS release actually only contain a version number bump commit?

Can that script use top-level await somehow?

Is it possible to include https://github.com/nodejs/node/pull/45659 in the 18.13 release?