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.

Sign up for GitHub

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

Jump to bottom

Update index.d.ts #5767

Open

hzgotb wants to merge 1 commit into knex:master from hzgotb:patch-1

hzgotb commented Dec 9, 2023

for below example will tell me 'this' implicitly has type 'any' because it does not have a type annotation.ts(2683)

const b = knex.select().from(function () {
        return this.select().as('a');
    }).toQuery();


          Update index.d.ts

d741631

fix missing type this in from

coveralls commented Dec 14, 2023

coverage: 92.787%. remained the same
when pulling d741631 on hzgotb:patch-1
into 4ca3dd5 on knex:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment