Skip to content
This repository has been archived by the owner on Aug 14, 2024. It is now read-only.

Remove comma-dangle #311

Open
cseas opened this issue Jan 1, 2023 · 1 comment
Open

Remove comma-dangle #311

cseas opened this issue Jan 1, 2023 · 1 comment

Comments

@cseas
Copy link

cseas commented Jan 1, 2023

typescript-eslint strongly recommends against using the "comma-dangle" rule:
https://typescript-eslint.io/rules/comma-dangle/

Since it is part of the original airbnb config, maybe we should just let the eslint one stay as it is instead and remove the override with the equivalent typescript-eslint rule.

@Kenneth-Sills
Copy link

Yup! With #354, since the Typescript rule was outright removed, this rule is being migrated to ESLint Stylistic. Once the upstream Airbnb config is updated, the override will be removed entirely.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants