dnlup comments

Results 54 comments of


                                            dnlup

Add the possibity to track a time execution of async functions

I am removing this one from the milestone. I would prefer not blocking a new release waiting for this feature.

Feat timing utils

I need to implement tests, but in the meantime this is the gist that I had in mind 😉.

Feat timing utils

> SMGT! GJ 🚀 > > Should we include some benchmarks to show overhead (if any) ? Yes, I'll work on it 😉

Feat timing utils

This will sit here for a while, I am also waiting on this PR: https://github.com/nodejs/node/pull/40625

Allow to specify a list of routes to ignore

I think for now we should focus on #62 , it should be enough for most cases without doing too many checks in the hooks. I am removing it from...

Allow to specify a list of routes to ignore

Let's keep the on hold now that we released #62 .

Evaluate the possibility of throwing in dynamic mode

Yeah, that's a good point.

Test that the package is packed correctly before publishing

That we could be forgetting to include a new file in the `files` field in the package.json 😁

Test that the package is packed correctly before publishing

I was thinking using npm pack and then installing the package in a dummy project to see if it doesn't crash.

WIP net: enable emitclose in socket

> @dnlup nit: I think it should be: > ~Ref: https://github.com/nodejs/node/issues/36636~ > Fixes: https://github.com/nodejs/node/issues/36636 Yup, thank you @RaisinTen