[META] TODOs

[asamuzaK]

See #41

Future TODOs

This code was not particularly type-safe from the start, so there's a myriad of type-casting and ignoring TS that's not ideal.

At some point we should:

• Ban ! cast usage #48
• Ban as type casting #49
• Fix export types to be more strict #51
• Remove object and any usage #50
• Move Array types to tuple types #52
• Refactor code that overrides existing variable types to dedicated single-use types #53
• Review tests to use assert instead of expect #47

Ref:

• TypeScript: Documentation - Do's and Don'ts
• Why doesn't type inference work on this interface: interface Foo<T> { }?

[asamuzaK]

@crutchcorn
One question.
When is .nvmrc used? Can we remove?

[crutchcorn]

Don't remove. It's used in CI

[asamuzaK]

Then is it okay to change value to lts/*?

[crutchcorn]

Truthfully idk. Lemme check and come back some time later

[asamuzaK]

Thanks, no need to hurry.

[asamuzaK]

Never mind about .nvmrc anymore.
I looked at the source and found that lts/* can be used without any problems.

• https://github.com/TanStack/config/blob/main/.github/setup/action.yml#L11
• https://github.com/actions/setup-node/blob/main/docs/advanced-usage.md#node-version-file
• https://github.com/actions/setup-node#supported-version-syntax

Thanks.