Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork6.3k
Add JSON linting#36192
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Add JSON linting#36192
Conversation
silverwind commentedDec 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I need to test whether it's possible to have this in one config file. Not super thrilled to add another config file to the repo root. Likely this would require passing explicit list of files because the |
silverwind commentedDec 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I conclude it's not possible to have a shared eslint config while retaining the ability to lint only JSON files without specifying them explicitely to eslint. So the two config files will have to remain. |
Uh oh!
There was an error while loading.Please reload this page.
Useshttps://github.com/eslint/json to lint all JSON and JSONC files in the repo.