Nick Rodriguez

Results 28 comments of Nick Rodriguez

There are two things that I think you're talking about: 1 would be adding `export` in the main typings of `node:readline`. Since readline re-exports the promise typings, to avoid a...

If blocking pr #61133 is gonna be held up for a bit, I'd like to move forward with code change reviews.

Rebased on top of master with the `no-const-enum` fixes.

Reverted lining changes and rebased on top of current master. @mcollina could you take a look at this again?

IMHO, TS is kinda about best practices, and its definitely best practice to use MV3. But that's pretty irrelevant here, because if we did support MV2, we couldn't support MV3...

I actually stopped using this action a while ago, so I'm not sure. I'll try replicating this again, and see what happens.

I've been summoned from my summer break grave, and oh boy did I just go on the tour of this repo. Looks like the original reason my package is breaking...

Thanks @jakebailey for fixing the ecosystem breaks, really appreciate it. I'm gonna recuse myself since this doesn't affect any of my packages anymore, but I'll keep the review up out...

Wait, I must have missed something my first time around. My reading now seems to imply that the undefined case ends with the Promise resolving with `undefined`, not the function...