You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I end up leaving these comments a lot on PRs. For each gap in coverage, at least one of the following is probably true:
The code is reachable, so a unit test should exercise it
The code can be refactored to not include that case
This is a difficult-to-represent edge case in types and a type assertion is warranted
Additional Info
Oftentimes when I post these it's on a lint rule that needs more RuleTester testing. But I don't want the version on my blog to be specific to lint rules - that stuff is intimidating to many. I'll probably suggest an equivalent blog post on the typescript-eslint blog for specifically lint rules.
💖
The text was updated successfully, but these errors were encountered:
Bug Report Checklist
main
branch of the repository.Overview
I end up leaving these comments a lot on PRs. For each gap in coverage, at least one of the following is probably true:
Additional Info
Oftentimes when I post these it's on a lint rule that needs more
RuleTester
testing. But I don't want the version on my blog to be specific to lint rules - that stuff is intimidating to many. I'll probably suggest an equivalent blog post on the typescript-eslint blog for specifically lint rules.💖
The text was updated successfully, but these errors were encountered: