Fixed some typescript errors

uwdata / arquero

Query processing and transformation of array-backed data tables.

https://idl.uw.edu/arquero/

BSD 3-Clause "New" or "Revised" License

1.29k stars 63 forks source link

Fixed some typescript errors #346

Closed munteannatan closed 2 months ago

munteannatan commented 6 months ago

I am researching the use of arquero in https://github.com/ni/nimble but installing the package will trigger some typescript errors when building.

I would like to not skip the lib check so I made some type fixes here.

Quite a few of them were too complex for me so I just marked them with // @ts-ignore.

Please let me know if you think there is a negative impact so I can update the changes.

jheer commented 2 months ago

Thank you! I appreciate all the work you've done here. The Arquero v6.0.0 updates (#358) include changes to improve TypeScript use, including explicit tests to check for TypeScript errors. So I'm closing out this PR in lieu of those updates.