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.

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

Fix Typings on DefinitelyTyped #104

Closed
4 tasks
MeirionHughes opened this issue Oct 9, 2021 · 7 comments
Closed
4 tasks

Fix Typings on DefinitelyTyped #104

MeirionHughes opened this issue Oct 9, 2021 · 7 comments
Assignees
Labels
maintenance General maintenance

Comments

@MeirionHughes
Copy link
Member

MeirionHughes commented Oct 9, 2021

  • encoding-down
    • default export no longer available
  • leveldown:
    • default export no longer available
    • getMany()
  • rocksdb:
    • default export no longer available
    • getMany()
  • levelup:
    • default export no longer available
    • getMany()
@MeirionHughes MeirionHughes self-assigned this Oct 9, 2021
@MeirionHughes MeirionHughes added the maintenance General maintenance label Oct 9, 2021
@sebastianst
Copy link

Cross-ref issue in levelup repo.

@sebastianst
Copy link

It would also be good to remove the unused type LevelUpBatch in the leveup types declaration file.

@MeirionHughes
Copy link
Member Author

been delayed on this ... I'm in the middle of a init product release that is draining my will to live. Once that's done, and the rocksdb gets a new release, I'll start upgrading our dependencies and work through the types.

@vweevers vweevers added this to To Do in Level (old board) Oct 29, 2021
@sebastianst
Copy link

The return type of db.iterator() also needs to be fixed. It should be something like AsyncIterable<[K, V]>. The current custom return type doesn't allow for async (const x of it) syntax in typescript code.

@sebastianst
Copy link

The LevelUpConstructor interface could be improved to accept a ErrorValueCallback<LevelUp> as the third parameter, it currently only receives ErrorCallback. (Or ErrorValueCallback<LevelUp|undefined>?)

@vweevers
Copy link
Member

vweevers commented Nov 9, 2021

Relevant: Level/abstract-level#5

@vweevers vweevers removed this from To Do in Level (old board) Dec 4, 2021
@vweevers
Copy link
Member

Closing because abstract-level and dependents have built-in type declarations. If someone wants to fix the typings of older modules on DT, go for it of course, but because @MeirionHughes is the main contributor of those types and seems to be busy, and the work needs to happen externally, I don't see an actionable item for us here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance General maintenance
Projects
Status: Done
Development

No branches or pull requests

3 participants