feat(plugin-eslint): support eslint.config.ts files

[cylewaitforit]

Adds support for eslint.config.ts files to the eslint plugin's default configuration.

https://eslint.org/docs/latest/use/configure/configuration-files#typescript-configuration-files

#955

[webpro]

Thanks for the PR, @cylewaitforit. There are some issues around loading such config files, mostly #818

Any chance you could add a test and a fixture with e.g. a eslint.config.ts file? Then we can better find out what works and what doesn't. Otherwise, we might be adding a feature that throws exceptions for many people new to the project which wouldn't improve the first impression.

[cylewaitforit]

@webpro I took a try at adding the fixture. Since it doesn't look like the eslint.config.js file has any related tests I am not really certain what to add in the eslint.tests.ts beside updating the total and processed. Please let me know if I am totally off the mark.

[webpro]

That's great for now, thanks!

Am I right in thinking that if you add something like eslint: ['eslint.config.ts'] to your project(s) that this works well for you already?

[pkg-pr-new]

Open in Stackblitz

npm i https://pkg.pr.new/knip@956

commit: 8f4d2f4

[cylewaitforit]

Yes, right now in my repo I’m using

 "eslint": {
    "entry": ["eslint.config.ts"]
  }

[webpro]

Thanks! I was thinking ahead a bit as I'm also trying to "promote" the eslint.config.* files to config files (to also fix other issues as I've linked to, including #806), but that requires a bit more work. This is one is complete, so happy to have merged this one. Thanks again.

[webpro]

🚀 This pull request is included in v5.45.0. See Release 5.45.0 for release notes.

Using Knip in a commercial project? Please consider becoming a sponsor.