Closed
Description
Before You File a Proposal Please Confirm You Have Done The Following...
- I have searched for related issues and found none that match my proposal.
- I have searched the current rule list and found no rules that match my proposal.
- I have read the FAQ and my problem is not listed.
My proposal is suitable for this project
- I believe my proposal would be useful to the broader TypeScript community (meaning it is not a niche proposal).
Link to the rule's documentation
https://typescript-eslint.io/rules/prefer-nullish-coalescing/
Description
This #10829 recent enhancement makes the rule check and recommend nullish coalescing for if() statements. IMO, this is not simpler to read, and won't include this to my organization's set of coding rules. The other recommendations from this rule are useful, and I'd like to keep it enabled, but this rule doesn't have a way to turn off the recommendation for the if() case specifically.
Please add an option to disable nullish coalescing checks for if() statements. Suggested name: ignoreIfStatements
Fail
// This code fails by default with the new version of this rule
if (!foo) {
foo = callAFunctionToGetAValue();
}
Pass
// This code should PASS if the new option (suggested name: ignoreIfStatements) is set to TRUE
if (!foo) {
foo = callAFunctionToGetAValue();
}
Additional Info
No response