Izaak Schroeder

Results 21 comments of Izaak Schroeder

No rush @briancavalier thanks for the feedback so far.

@briancavalier any updates? πŸ˜„

@GeoffreyBooth @JakobJingleheimer I updated my PR to address this issue πŸ˜„ I followed the proposal from @giltayar and the signature for `register` is updated to be able to pass arbitrary...

Done! PR is here: https://github.com/izaakschroeder/node/pull/1 I'm not sure how I can put the PR on node's repo without the base branch also living in node's repo πŸ€” Would you prefer...

@giltayar maybe? I can help… I built much of the new `register` machinery so perhaps I could try to build out an example that fits here? #### `testdouble-super-register.js` ```ts import...

For what it's worth I'm also not against exposing a `getLoaders()` type method on `node:module`. I don't think it would be hard to implement and we could have it store...

> Would we want to expose the whole registry or would just name and position do the job? Would someone want to use the loader or just know about it?...

Interesting πŸ€” I'd love to learn a bit more. Would it be possible to point out the area in the code that is responsible for this behaviour? My mental model...

Ah excellent thanks @aduh95 πŸ˜„ I'll have a look in that file!

@aduh95 thanks for pointing me in the right direction πŸ˜„ I'm not sure if this approach is sensible or not but I have a working patch now that solves the...