-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Add PSQLError debugDescription #372
Conversation
…ance to PostgresQuery, PostgresBindings.Metadata, and PostgresBindings.
…ror.debugDescription`. Remove workaround for missing ByteBuffer conformance (it's not missing anymore).
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.
Looks great. Now we need some tests!
…ror descriptions. Several tweaks to PostgresQuery and PostgresBindings descriptions to make them a bit less unwieldy, since devs are likely to see them a lot even if end-users don't.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #372 +/- ##
==========================================
+ Coverage 42.98% 44.35% +1.36%
==========================================
Files 118 118
Lines 8243 8629 +386
==========================================
+ Hits 3543 3827 +284
- Misses 4700 4802 +102
|
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.
Thanks for the progress on this. A few comments
Fixes #354, #380. Continues work from #356 and #360.
Requires minor version bump (adds new public API in the form of protocol conformances).