Skip to content

🔪 Deprecate no-loss-of-precision rule extension #7277

Closed
@JoshuaKGoldberg

Description

@JoshuaKGoldberg

Overview

https://typescript-eslint.io/rules/no-loss-of-precision states that it exists to add support for numeric separators to the base eslint/no-loss-of-precision rule. But, numeric separators have passed stage 3 and are natively supported in ESLint I believe. [Example ESLint playground with the rule working on a number with a numeric separator]

tl;dr: I propose we remove the rule extension because the base rule contains all its features.

👉 I'm not proposing we remove the base rule (or ask ESLint to)

Whether it exists in popular configs such as eslint-config-airbnb-typescript and eslint-config-standard-with-typescript

Sourcegraph queries showing how often it appears in user configs

Sourcegraph query showing it used in <200 repos: this doesn't seem like a particularly common rule for users to explicitly specify. That makes sense to me given that it's in the base recommended config (and has been for a while).

Metadata

Metadata

Assignees

No one assigned

    Labels

    accepting prsGo ahead, send a pull request that resolves this issuelocked 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-pluginrepo maintenancethings to do with maintenance of the repo, and not with code/docs

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions