Bryan English
Bryan English
Just chiming in to say that while I was (correctly) moved to the emeritus list, I intend to become more active again, and would like to resume release work. To...
> Hopefully, Node.js 20.x breaking your tools is another argument you can bring to your boss why you should be more involved in the dev process, and allocating resources to...
FWIW my benchmark was effectively: ```js const origRequire = Module.prototype.require for (let i = 0; i < SOME_BIG_ENOUGH_NUMBER; i++) { require('dd-trace/packages/datadog-instrumentations') startTimer() require('undici') stopTime() clearCache() Module.prototype.require = origRequire } ```...
This PR needs to be completely redone. Closing.
This just needs a re-do, if we're going to do it.
@bm1549 It looks like yes, so I'll close this. @ida613 please feel free to re-open if that's not the case.
Note that this now removes support (i.e. no longer instruments) Next.js `>=12.0.0
@tlhunter I've added that in the PR description. Since this should only need to be done once, I don't think it warrants being included elsewhere.
Moving this to draft because it needs quite a huge rebase, or potentially a re-do.
A different project is tackling this.