lrud icon indicating copy to clipboard operation
lrud copied to clipboard

[Maintenance] Replace all the `any` with TS types

Open epatrasc opened this issue 6 years ago • 2 comments

Is your feature request related to a problem? Please describe.

We have lots of "any" as type, we should fix this.

Describe the solution you'd like A clear and concise description of what you want to happen. Add the correct type for all the any

epatrasc avatar Nov 08 '19 10:11 epatrasc

Have started a branch with tsc in strict mode to try to resolve some of this: https://github.com/bbc/lrud/tree/strict-types

edmorrish avatar Nov 08 '19 16:11 edmorrish

Types are extremely messy across the board to be honest, almost everything is nullable or undefined at all times so it'd probably be a good idea to do a review of the Lrud data structure and come up with some sensible defaults for the constructor.

edmorrish avatar Nov 08 '19 16:11 edmorrish