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.

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

[Docs]: Remove jsdoc comment about exposeQueryBuilder #163

Open
akatechis opened this issue Oct 31, 2022 · 1 comment
Open

[Docs]: Remove jsdoc comment about exposeQueryBuilder #163

akatechis opened this issue Oct 31, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@akatechis
Copy link

Operating System

Windows 11

Deno version

1.27.0

Nessie version

2.0.8

Bug description

The JSDoc comment for Info suggests there's an exposeQueryBuilder option that can be used to inject a query builder into migrations, but no such option exists on NessieConfig.

Steps to reproduce

See docs

Aditional information

I think I would actually prefer if the query builder was indeed included, as it would make it easier to construct migrations, but if that goes against the philosophy of Nessie, then it should have this comment removed to save people time from looking through docs for something that doesn't exist :)
@akatechis akatechis added the bug Something isn't working label Oct 31, 2022
@halvardssm
Copy link
Owner

Hi @akatechis ! Thank you for the bug report, but please see #165.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@akatechis @halvardssm