denyncrawford / dndb

A Deno πŸ¦• persistent, embeddable and optimized NoSQL database for JS & TS

Home Page:https://dndb.crawford.ml

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

type errors with any's

twilson63 opened this issue Β· comments

Hello πŸ‘‹

Neat Project! I am looking for something of a lite nosql db for small docs and found this project.

When I went to work with it using typescript, I ran into a few type warnings with the any type.

So I added some types to the src/mod.ts file, would you be interested in reviewing and see it this is worthy of a PR?

If you get a chance check it out. - https://github.com/hyper63/dndb/blob/main/src/mod.ts

It did make my type errors go away, but I had to use ts-ignore, which I know is not great, I will have to look into better solutions to add types to the executor add function, since it may return different results based on the initial argument.

Thanks again for the project, if this change is in the direction you are looking for, let me know and I will submit a PR.

Cheers!

Hi dude πŸ‘‹ , I was checking and I see you got rid of the ts-ignore, amazing! So when you have all the changes ready, list them in the PR to review and merge them. Thanks for getting involved with DnDB, big changes are coming.

Done, this will be available at 0.3.3 this Monday, thank you mate!