Bryan English

Closing this since it's ancient. Please feel free to comment again if you're still seeing this, particularly if there's any new information.

Hi @flavienliger! I know it's been about 6 years since you submitted this PR, but if you'd still like it to be mergd there's a couple of things it's going...

@dyladan > When the API changes I assume IITM will be broken. The question becomes, in what way is it broken? Modules will fail to load and apps will likely...

Suggestion on this one: Use a list or a set for each room. You don't actually need to grab each message by unique ID, since this is just for history,...

Additional notes: I attempted with the following changes, and it still happens: * Replace the async_hook with a promise hook. This narrows it down to promise hooks, I think. *...

There are a few things needed here. Off the top of my head: * [ ] Testing in Deno in CI * [ ] Testing that things currently dependent on...

@devsnek now that dd-trace uses AsyncLocalStorage, we can close this, right?

Overall I see a bunch of wins in doing this roughly similar to how Deno did it. 1. There would be less non-core code needed for OTel-style observability. This is...

@simon-id yep that's right. I also had to update eslintrc, since there's a new mocha rule to disable.

Closing. This is _already_ out of date.