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

Switch to @eslint/json #1749

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

ericcornelissen
Copy link
Owner

Summary

Switch to this "officially supported" alternative following https://eslint.org/blog/2024/10/eslint-json-markdown-support/. (Note that this project does not use a MarkDown plugin for ESLint.)

Following [1] this replaces `eslint-plugin-jsonc` by `(at)eslint/json`
for linting JSON(C) in this project. The motivation for this change is
supported by the FAQ of the new package.

The reduction in rules is further motivated by ESLint's general move
away from formatting-like rules to lint-like rules, which I believe
makes sense for the present project given the use of Prettier.

--
1. https://eslint.org/blog/2024/10/eslint-json-markdown-support/
@ericcornelissen ericcornelissen added the dependencies Changes to the project's dependencies label Oct 19, 2024
@github-actions github-actions bot added the meta Relates to the project or repository itself label Oct 19, 2024
@ericcornelissen
Copy link
Owner Author

This is blocked by eslint/json#41 (@eslint/json does not work with the dangling comma but prettier automatically adds it).

@ericcornelissen ericcornelissen added the pending This cannot be worked on yet label Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Changes to the project's dependencies meta Relates to the project or repository itself pending This cannot be worked on yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant