-
Notifications
You must be signed in to change notification settings - Fork 512
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
feat: add mysql2 responsehook #915
Conversation
Codecov Report
@@ Coverage Diff @@
## main #915 +/- ##
==========================================
+ Coverage 95.91% 96.10% +0.19%
==========================================
Files 13 16 +3
Lines 856 1001 +145
Branches 178 204 +26
==========================================
+ Hits 821 962 +141
- Misses 35 39 +4
|
@nozik I think this is ready to be merged. |
@blumamir Thanks, it's now rebased. For some reason I'm not seeing the option in the link you've shared, I discussed that with Rauno as well. Anyway, feel free to merge. |
Thanks! sorry, I just merged another PR to main, can you please rebase again? |
Which problem is this PR solving?
responseHook
, that enables collecting the query result.Short description of the changes
responseHook
config, and add tests to verify its behavior.Checklist
npm run test-all-versions
for the edited package(s) on the latest commit if applicable.