-
-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Instrument postgresql simple #11
base: main
Are you sure you want to change the base?
Conversation
Obviously this isn't done yet, but is it moving in a usable direction, @iand675 ? |
fd0e412
to
65898ad
Compare
Finally got around to finish it up so it's ready for review. |
65898ad
to
3028a4a
Compare
instrumentation/postgresql-simple/src/OpenTelemetry/Instrumentation/PostgresqlSimple.hs
Outdated
Show resolved
Hide resolved
instrumentation/postgresql-simple/src/OpenTelemetry/Instrumentation/PostgresqlSimple.hs
Outdated
Show resolved
Hide resolved
Thanks for pushing this along! Really excited to land this. |
974c1c3
to
9ab228e
Compare
Hopefully this addresses all issues. |
@@ -39,6 +39,7 @@ library | |||
, postgresql-libpq | |||
, postgresql-simple | |||
, text | |||
, unliftio-core |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@magthe please edit package.yaml instead of this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm hoping the change I pushed addresses this.
9ab228e
to
46729f5
Compare
Ping! |
This has lingered for a while now. Will it be merged if I resolve the conflicts? |
Sharing this early to get feedback and suggestions.
Attempt to address #7.