search

nextapps-de / flexsearch

Next-Generation full text search library for Browser and Node.js
Apache License 2.0
12.53k stars 491 forks source link

Types rewrite #254

Closed ndrewtl closed 3 years ago

ndrewtl commented 3 years ago

The types as written in index.d.ts appear to have been wrong-- the declared type Index did not coincide with the actual class FlexSearch.Index. Therefore this PR lays down the types correctly, at least for that one class. The types representing WorkerIndex and Document have not been written, it seems, and so those should be handled separately in the future. Because the three types have much in common, we might consider making a common interface that is then implemented by all three.

ndrewtl commented 3 years ago

Oh shoot, sorry. This was intended to be merged into the skiff-org repository, not this one. I might make a similar PR including these types in the future for this repo. But for now I'm just doing it in my own. Closing.