Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Enhancement: [prefer-nullish-coalescing] recognize "||=" #5229

Closed
@jakebailey

Description

@jakebailey

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

In cases where an assignment is made to a nullish variable via ||=, it'd be nice if ??= was suggested instead.

Fail

let foo: Map<string, string> | undefined;

foo ||= new Map();

foo = foo || new Map(); // These are already handled in prefer-nullish-coalescing.
(foo || new Map()).get("key");

Pass

let foo: Map<string, string> | undefined;

foo ??= new Map();

foo = foo ?? new Map(); // These are already handled in prefer-nullish-coalescing.
(foo ?? new Map()).get("key");

Additional Info

No response

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