-
Notifications
You must be signed in to change notification settings - Fork 98
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
Support Knex #468
Merged
DominicKramer
merged 14 commits into
googleapis:master
from
DominicKramer:feature/support-knex
May 23, 2017
Merged
Support Knex #468
DominicKramer
merged 14 commits into
googleapis:master
from
DominicKramer:feature/support-knex
May 23, 2017
Commits on May 13, 2017
-
Configuration menu - View commit details
-
Copy full SHA for 698af35 - Browse repository at this point
Copy the full SHA 698af35View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b9b5da - Browse repository at this point
Copy the full SHA 6b9b5daView commit details -
Fix context loss in knex without patching bluebird
I think this patch is specific to mysql. I will investigate this further.
Configuration menu - View commit details
-
Copy full SHA for a7c6bdf - Browse repository at this point
Copy the full SHA a7c6bdfView commit details -
Configuration menu - View commit details
-
Copy full SHA for f80681f - Browse repository at this point
Copy the full SHA f80681fView commit details -
Intercept the module root to patch
transaction
This commit intercepts the module root to patch the `transaction` method. This works because the `knex` module exports a constructor and the object that the constructor constructs has a `client` property that is the client being used. This client can have its `transaction` method updated in a way to preserve context.
Configuration menu - View commit details
-
Copy full SHA for aa7279a - Browse repository at this point
Copy the full SHA aa7279aView commit details
Commits on May 16, 2017
-
Configuration menu - View commit details
-
Copy full SHA for cead626 - Browse repository at this point
Copy the full SHA cead626View commit details
Commits on May 18, 2017
-
Configuration menu - View commit details
-
Copy full SHA for 93b2db2 - Browse repository at this point
Copy the full SHA 93b2db2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4b041f7 - Browse repository at this point
Copy the full SHA 4b041f7View commit details -
Configuration menu - View commit details
-
Copy full SHA for fe7dc68 - Browse repository at this point
Copy the full SHA fe7dc68View commit details -
Configuration menu - View commit details
-
Copy full SHA for cdd2c11 - Browse repository at this point
Copy the full SHA cdd2c11View commit details -
Configuration menu - View commit details
-
Copy full SHA for 74c46ab - Browse repository at this point
Copy the full SHA 74c46abView commit details -
Configuration menu - View commit details
-
Copy full SHA for c74a04a - Browse repository at this point
Copy the full SHA c74a04aView commit details
Commits on May 22, 2017
-
Configuration menu - View commit details
-
Copy full SHA for 19663ce - Browse repository at this point
Copy the full SHA 19663ceView commit details
Commits on May 23, 2017
-
Configuration menu - View commit details
-
Copy full SHA for ec1d9c0 - Browse repository at this point
Copy the full SHA ec1d9c0View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.