-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
json language conflict with jsonc #11
Comments
I noticed that too. However, that's the settings zed file, so I suppose zed does the same thing as VSCode. If you opened another JSON file, you wouldn't have these errors. I suppose we'll have to add a new expectation |
To address potential future language-detection problems, we may want to support using filepath glob patterns and the first line of the file to detect languages just like what VSCode does. We can also learn it from other editors or tools like Prettier. |
Yes, but zed actually implies all json files are jsonc, not just the settings files. |
FWIW i'm getting similar errors in The error is really confusing because unlike the settings file it doesn't make sense at first, but I suppose the comments later screw it up so that's why the whole thing is highlighted red. |
So zed has no specific language for .jsonc files, all json files are interpreted as jsonc under "JSON".
Now that the biome extensions uses "json" to interpret json files, we get this stuff:
We should probably use jsonc for all json files. Though not sure who is right here.
The text was updated successfully, but these errors were encountered: