Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

💡 (ESLint plugin): Support eslint.config.ts #859

Closed
taro-28 opened this issue Nov 29, 2024 · 2 comments
Closed

💡 (ESLint plugin): Support eslint.config.ts #859

taro-28 opened this issue Nov 29, 2024 · 2 comments
Labels
feature request Feature request

Comments

@taro-28
Copy link
Contributor

taro-28 commented Nov 29, 2024

Suggest an idea for Knip

Hello!

ESLint now supports using TypeScript configuration files like eslint.config.ts as an experimental feature. I’d like to see support for this in the Knip ESLint Plugin as well!

I believe this can be handled simply by updating the entry as follows!

const entry = ['eslint.config.{js,cjs,mjs}'];

// Add ts extensions
const entry = ['eslint.config.{js,cjs,mjs,ts,cts,mts}'];

Would it be okay if I created a PR for this?

@taro-28 taro-28 added the feature request Feature request label Nov 29, 2024
@taro-28 taro-28 changed the title 💡 (ESLint plugin): Support eslint.config.ts 💡 (ESLint plugin): Support eslint.config.ts Nov 29, 2024
@webpro
Copy link
Collaborator

webpro commented Nov 29, 2024

I'd love that but there are things holding us back: this is discussed in #818

#806 is also blocked by this

Should add that I didn't check up on the status of things recently.

@taro-28
Copy link
Contributor Author

taro-28 commented Nov 30, 2024

Got it, I see I missed checking the existing issues…!
I'll close this now and look forward to it being supported someday!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Feature request
Projects
None yet
Development

No branches or pull requests

2 participants