-
-
Notifications
You must be signed in to change notification settings - Fork 339
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.config.js
support
#3217
Comments
The error message is leading, it should be something like "we do not support files other than json". |
@Boshen yes, but support for |
@dalisoft please refer to this statement @Boshen made: #2214 (comment) |
This is actually a HUGE issue because now the new "Flat Config" format (the |
Do you have a rough idea of how to make it work? i.e. load eslint and get all the information we need? |
@Boshen if you'd like I can put something together |
You can't load and evaluate JavaScript from Rust, that's the crux of the problem. To make this even remotely work, you would need to execute a |
I'm not worried about the performance impact, I'm worried about the maintenance burden put into this project, as stated in #2214 (comment) When I was doing research while setting up oxlint, I read almost all ESLint related issues from GitHub, stackoverflow, discord, reddit etc, and 90% of them are configuration issues, and a lot of them are unanswered due to its context and complexity. |
Maybe we can use |
If oxlint is going to have compatibitly only with eslint 8 and before configs (I think a valid opinon given the burden). Perhaps oxlint needs its own new config format that would have the same advantages of the eslint flat config. There is a lot of great information here on why the changes were made: https://eslint.org/blog/2022/08/new-config-system-part-1/ Perhaps a similar system but written in static RON files would be a good solution? Oxlint could then support both the past eslint json format and its own new format |
Currently
eslint.config.js
is not supported as tested bybunx oxlint . --import-plugin --config=eslint.config.js
And got
The text was updated successfully, but these errors were encountered: