Skip to content

Enhancement: [prefer-optional-chain] support if (x != null) case #10842

Open
@kirkwaiblinger

Description

@kirkwaiblinger

Before You File a Proposal Please Confirm You Have Done The Following...

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

Followup to #10137 (comment) - in addition to the if (x) case covered by #6309#10137, let's also cover the if (x != null) case.

Fail

declare const callable: () => void | null;

if (callable != null) {
  callable();
}

Pass

declare const callable: () => void | null;

callable?.();

Additional Info

Probably best to consider this blocked on #10137

Metadata

Metadata

Assignees

No one assigned

    Labels

    accepting prsGo ahead, send a pull request that resolves this issueenhancement: plugin rule optionNew rule option for an existing eslint-plugin rulepackage: eslint-pluginIssues related to @typescript-eslint/eslint-plugin

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions