Use typescript-eslint's recommended configs as a starting base #114
JoshuaKGoldberg
started this conversation in
Ideas
Replies: 1 comment
-
|
For reused configs like this, a common rollout strategy I default to is:
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
👋 Hi! I'm one of the typescript-eslint maintainers, and I see the custom list of rules in
/packages/config/src/eslint/typescript.js. It's great to be so intentional about the lint settings.But - there are also shared configs that come with the typescript-eslint plugin: https://typescript-eslint.io/users/configs. Those configs -especially
recommendedandrecommended-type-checked- enable the commonly useful rules that most projects generally find useful and not too opinionated.Our recommendation is that projects start with those presets and add customizations on top of them as needed. This has a few benefits:
typescript.js, 12 have the same options asrecommendedTypeChecked.Specifically, those repeated rules are...
method-signature-styleno-duplicate-enum-valuesno-extra-non-null-assertionno-for-in-arrayno-misused-newno-namespaceno-non-null-asserted-optional-chainno-unnecessary-type-assertionprefer-as-constprefer-for-ofrequire-awaittriple-slash-referenceProposal: would you be open to switching
eslint-configto be based ontseslint.configs.recommendedTypeCheckedandtseslint.configs.stylisticTypeChecked?If it helps, I'd be happy to drive this work forward myself! It directly contributes to typescript-eslint/typescript-eslint#9141, which I'm working on now to prep for our v8 release.
Beta Was this translation helpful? Give feedback.
All reactions