-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Description
Overview
An option named allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing exists in three rules:
Each comes with a very stern explanation of why not to use it:
Without
strictNullChecks, TypeScript essentially erasesundefinedandnullfrom the types. This means when this rule inspects the types from a variable, it will not be able to tell that the variable might benullorundefined, which essentially makes this rule a lot less useful.
It's bugged me for a while that we've kept these options around. They add bloat to the docs and rule logic. And practically nobody uses them.
A Sourcegraph search for allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing shows only 3 public repos visibly using setting it to true - and 5 setting it to false.
Proposal: let's...
- Deprecate the options as a non-breaking change now
- Remove them in the next major version with functional changes
π