Skip to content
This repository was archived by the owner on Mar 7, 2025. It is now read-only.

Conversation

sxzz
Copy link
Collaborator

@sxzz sxzz commented Oct 21, 2022

@sxzz sxzz requested a review from Shinigami92 as a code owner October 21, 2022 07:18
@Shinigami92 Shinigami92 added the enhancement New feature or request label Oct 21, 2022
@Shinigami92
Copy link
Member

@sxzz I'm not sure about the term legacy 🤔
Is it coming from ESLint? Or did you came up with this term?

@sxzz
Copy link
Collaborator Author

sxzz commented Oct 21, 2022

@Shinigami92 According to the blog, I think flat config is the future. Actually, I'm not sure, but it has to get a name. What name do you prefer?
legacy, nested, or something else?

@Shinigami92
Copy link
Member

@Shinigami92 According to the blog, I think flat config is the future. Actually, I'm not sure, but it has to get a name. What name do you prefer? legacy, nested, or something else?

Ah yeah, then I'm against legacy as it indicates now as it would be deprecated, but it is not. The new FlatConfig is just the new way for now.

For now I wouldn't change it at all and just stick with config and flat-config and use this terminology the whole way down.
Like ESLintConfig and FlatESLintConfig and so on.

... And oh yeah, please use FlatESLintConfig instead of ESLintFlatConfig as this is what they call it:

image

@sxzz
Copy link
Collaborator Author

sxzz commented Oct 21, 2022

@Shinigami92 I agree with you. I will change it

@Shinigami92
Copy link
Member

I have some Meetings and Company Work today, if I find the time this evening, I will release your (welcome 🤗) additions 👍
(I'm in German TimeZone)

@sxzz sxzz requested a review from Shinigami92 October 21, 2022 07:53
@sxzz
Copy link
Collaborator Author

sxzz commented Oct 21, 2022

@Shinigami92 OK thanks~

@Shinigami92 Shinigami92 merged commit ff6bd71 into eslint-types:main Oct 23, 2022
@sxzz sxzz deleted the feat/flat branch October 23, 2022 10:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support new ESLint config
2 participants