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 languages.json to JSON5 #983

Closed
XAMPPRocky opened this issue Apr 18, 2023 · 0 comments · Fixed by #986
Closed

Switch languages.json to JSON5 #983

XAMPPRocky opened this issue Apr 18, 2023 · 0 comments · Fixed by #986

Comments

@XAMPPRocky
Copy link
Owner

Currently we're using a JSON parser to parse the contents of languages.json however this format is pretty unfriendly to contributors, with onerous requirements like no trailing commas (#982).

I'd like to move the parsing to using JSON5 as that is compatible with the current format, so it doesn't require a big change, while also providing a lot QoL improvements like trailing commas and comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant