Level / level

Universal abstract-level database for Node.js and browsers.
MIT License
1.55k stars 106 forks source link

types: add typings #154

Closed ChristianMurphy closed 5 years ago

ChristianMurphy commented 5 years ago

resolves https://github.com/Level/level/issues/120

vweevers commented 5 years ago

Typings for Level are maintained in DT.

cryptoquick commented 5 years ago

I actually don't see those in DefinitelyTyped.

yarn add @types/level returns not found.

vweevers commented 5 years ago

@cryptoquick I'll rephrase: typings for projects in the Level org are (or should be) in DT.

ethernomad commented 4 years ago

Yeah @types/level doesn't exist. This needs to be created.

cryptoquick commented 4 years ago

Yeah, it'd probably be a good idea to make this a priority, given how popular TypeScript is. I was just looking at Level the other day. Does anyone have any other solution for this other than just copying and pasting the files in this PR into your own project? If the devs here aren't taking this as a priority, that's also a problem.

vweevers commented 4 years ago

If the devs here aren't taking this as a priority, that's also a problem.

The devs here are mostly JS users, we don't maintain typings. We tried to, but it quickly became painful and folks from DT recommended that we move the typings there. That move implicitly included the responsibility of maintenance, from us to the community.

Bottom line, if you want a @types/level package, please open a pull request to DT. We're happy to answer any JS questions though, for example if you need clarification on the behavior of some specific interface. Good luck!