-
Notifications
You must be signed in to change notification settings - Fork 26
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
Knex update #43
base: master
Are you sure you want to change the base?
Knex update #43
Conversation
Knex returns native JS promises instead of Bluebird ones. This means that you no longer use such methods as `map`, `spread` and `reduce` on QueryBuilder instance.
Anyone? |
I was about to make a PR for this also, we are going to have to fork in the meantime. |
Updated to knex 2.3.0 |
# Conflicts: # package-lock.json
Updated to knex 2.4.2 |
Can somebody please review my pull request? |
I don't really have any domain knowledge here...we just pointed our I will say we did that 9+ months ago and haven't run into any issues with the stuff we are doing (mainly raw query building). |
Could anyone have a look at this? We are also running our tests against this branch for a couple of monthes. |
Updates knex to 2.1.0 and fixes some issues