Skip to content
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

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Knex update #43

wants to merge 17 commits into from

Conversation

HaTiIn
Copy link

@HaTiIn HaTiIn commented Jul 4, 2022

Updates knex to 2.1.0 and fixes some issues

Hans Thielen added 2 commits July 5, 2022 10:16
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.
@HaTiIn
Copy link
Author

HaTiIn commented Aug 22, 2022

Anyone?

@spilliton
Copy link

I was about to make a PR for this also, we are going to have to fork in the meantime.

@HaTiIn
Copy link
Author

HaTiIn commented Sep 29, 2022

Updated to knex 2.3.0

@inform-aschneid
Copy link

Updated to knex 2.4.2

@HaTiIn
Copy link
Author

HaTiIn commented Feb 21, 2023

Can somebody please review my pull request?

@schmetzyannick
Copy link

@spilliton ?

@spilliton
Copy link

I don't really have any domain knowledge here...we just pointed our package.json to this PR/fork.

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).

@schmetzyannick
Copy link

Could anyone have a look at this? We are also running our tests against this branch for a couple of monthes.
@emurphy ?

@schmetzyannick schmetzyannick requested a review from a team as a code owner September 5, 2023 08:54
@schmetzyannick schmetzyannick requested review from kklein-deliverr and removed request for diogenes-deliverr, stewartb-deliverr, udit-deliverr and a team September 5, 2023 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants