Brett McBride
Brett McBride
I'm also interested in updating the driver to work with api v4 if somebody has the time to discuss and review...
cool, does this repo need a deprecation notice in the readme? I found my way here via npm (but also found links here in the MM documentation)...
I've started playing with this, and success has been dependent on how DBMS vendors represent exception codes. For example, postgres uses the sql standard [SQLSTATE error codes](https://www.postgresql.org/docs/current/static/errcodes-appendix.html#ERRCODES-TABLE) which are non-numeric,...
@chukShirley I could probably write a failing integration test for that to demonstrate a current bug. Can you provide a simple query which would cause a non-integer error code from...
Currently, when running phan on php 8.0+ when supplying the second parameter and using the protobuf extension (but not the native library), this results in a `PhanParamTooManyInternal Call with 2...
Looks like a sensible change to me. Can you take a look at the phan issue? It might just need to be suppressed...
@alecsammon progress, now some phpstan issues. FYI you can run all of the CI checks locally: `PHP_VERSION=8.3 PROJECT=Instrumentation/Laravel make all`
I'm wondering whether attaching logs as trace events is the right thing to do at all? Is there any precedent? Otel has a logging signal, and we already have contrib...
If the pre hook doesn't start+activate a span, then the post hook could potentially deactivate some other scope, couldn't it? If so, might it be safer to create, activate and...
> Would you be willing to open a PR? Yes. I'll start to play around with it to see how I can achieve something basic first. I like the idea...