Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Description
Overview
An option namedallowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing
exists in three rules:
Each comes with a very stern explanation of why not to use it:
Without
strictNullChecks
, TypeScript essentially erasesundefined
andnull
from the types. This means when this rule inspects the types from a variable,it will not be able to tell that the variable might benull
orundefined
, 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 forallowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing
shows only 3 public repos visibly using setting it totrue
- and 5 setting it tofalse
.
Proposal: let's...
- Deprecate the options as a non-breaking change now
- Remove them in the next major version with functional changes
💖