Skip to content

Enhancement: [restrict-plus-operands] Enable checkCompoundAssignments by default #6851

@JoshuaKGoldberg

Description

@JoshuaKGoldberg

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/restrict-plus-operands

Description

Right now, checkCompoundAssignments defaults to false (off). It's been like that since August 2019, in @a-tarasyuk's #892. Turning it on back them would have been a breaking change.

Since we've got a v6 coming up soon, how about turning it to true (on) by default?

Even better: how about renaming to skipCompoundAssignments, so that we don't have a confusing truthy-by-default value?

Additional Info

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    accepting prsGo ahead, send a pull request that resolves this issuebreaking changeThis change will require a new major version to be releasedenhancement: 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

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions