-
Notifications
You must be signed in to change notification settings - Fork 25
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
🛠 Tooling: replace jsonc-eslint-parser
with @eslint/json
#655
Comments
Yeah 😞. As much as I'd love to use the new first-party package, I don't think it's time yet to drop support for ESLint 8. Once other popular plugins start to drop support then I think we can mark this as unblocked. Thanks for filing! |
Hi there. I've created an experimental tool that converts You can find a working example online here. |
That's fantastic, thanks @ota-meshi! Marking as accepting PRs to switch this package to using the compat utils. Note that doing so is probably not a trivial change. It'll probably touch most or all of the rules in the plugin. |
@all-contributors please add @rakleed for tool.
|
I've put up a pull request to add @rakleed! 🎉 |
Adds @rakleed as a contributor for tool. This was requested by JoshuaKGoldberg [in this comment](#655 (comment)) --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Bug Report Checklist
main
branch of the repository.Overview
As suggested in #601 (comment), I am creating a separate issue to discuss replacing
jsonc-eslint-parser
dependency with@eslint/json
.@eslint/json
is smaller and uses fewer dependencies thanjsonc-eslint-parser
. The announcement post also says that it can be used as a parser for other plugins.The main problem is that the plugin only works since version 9.6.0 of ESLint, and you can use only a flat config. So you will have to stop supporting eslint@8, but now it still has more weekly downloads than eslint@9.
Additional Info
No response
The text was updated successfully, but these errors were encountered: