Wes Todd
Sorry should have fully read the PR description before posting a review. > The complex regex tests expect the span names to contain '/test/arr/:id,/\\/test\\/arr[0-9]*\\/required(path)?(\\/optionalPath)?\\/(lastParam)?/' but a couple of them instead...
> One option would be to emit tracing events directly from express via diagnostics_channel now that Node version support would allow this. This would mean no more relying on internals!...
With v5 moving to `latest`, it would be great to land this for the folks asking. Hate to be another `+1` comment, but it is unclear if there is something...
I think the medium to long term goal is to remove the template engines from "core" express and move things over to middleware (details TBD I think). I don't know...
Other than number 4, I think each of those could land as individual PRs. If you could open a PR or a new issue to discuss what new features you...
No worries on delays! It is all volunteer work, so no expectations at all. I understood the line item I think, and my concern was just that it is usually...
I have a lot of thoughts on this, and only 7 minutes before I have a meeting. I just wanted to drop my first comment: I think there are a...
Also not covered in @ctcpip's notes from the meeting: @kjugi is planning on opening an RFC/ADR in this repo to address this in so we have a doc about the...
Fixed in #190
Time to post all those AI generated logs?