Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueenhancement: plugin rule optionNew rule option for an existing eslint-plugin rulelocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.package: eslint-pluginIssues related to @typescript-eslint/eslint-plugin
Description
Before You File a Proposal Please Confirm You Have Done The Following...
- I havesearched for related issues and found none that match my proposal.
- I have searched thecurrent rule list and found no rules that match my proposal.
- I haveread 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 isnot 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 ruleif(!foo){foo=callAFunctionToGetAValue();}
Pass
// This code should PASS if the new option (suggested name: ignoreIfStatements) is set to TRUEif(!foo){foo=callAFunctionToGetAValue();}
Additional Info
No response
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueenhancement: plugin rule optionNew rule option for an existing eslint-plugin rulelocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.package: eslint-pluginIssues related to @typescript-eslint/eslint-plugin