node-wot icon indicating copy to clipboard operation
node-wot copied to clipboard

Enable `@typescript-eslint/no-unnecessary-condition` rule

Open JKRhb opened this issue 1 year ago • 1 comments

In the context of #1176, I noticed that we currently perform a lot of checks and nullish coalescing that are actually not really necessary based on our current typings. In some cases, this makes the code less readable, in other cases this can pave the way for potential bugs since some possible null values are hidden behind typecasts.

eslint provides the rule @typescript-eslint/no-unnecessary-condition which makes it possible to detect and fix these problems. As another step toward more type safety, this rule could be added on a per-package basis again, which should ultimately make the library more reliable.

JKRhb avatar Nov 29 '23 13:11 JKRhb