-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuedocumentationDocumentation ("docs") that needs adding/updatingDocumentation ("docs") that needs adding/updatinglocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.
Description
Before You File a Documentation Request Please Confirm You Have Done The Following...
- I have looked for existing open or closed documentation requests that match my proposal.
- I have read the FAQ and my problem is not listed.
Suggested Changes
typescript-eslint/packages/eslint-plugin/docs/rules/no-unnecessary-condition.mdx
Lines 15 to 20 in 7469e72
The following expressions are checked: | |
- Arguments to the `&&`, `||` and `?:` (ternary) operators | |
- Conditions for `if`, `for`, `while`, and `do-while` statements | |
- `case`s in `switch` statements | |
- Base values of optional chain expressions |
We also check things like array predicate return types and type predicates now
Affected URL(s)
https://typescript-eslint.io/rules/no-unnecessary-condition
Additional Info
We should either remove this list or update it to be accurate
Metadata
Metadata
Assignees
Labels
accepting prsGo ahead, send a pull request that resolves this issueGo ahead, send a pull request that resolves this issuedocumentationDocumentation ("docs") that needs adding/updatingDocumentation ("docs") that needs adding/updatinglocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.