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

JSON5 support #126

Open
Mechite opened this issue Nov 9, 2022 · 3 comments
Open

JSON5 support #126

Mechite opened this issue Nov 9, 2022 · 3 comments
Assignees

Comments

@Mechite
Copy link

Mechite commented Nov 9, 2022

Self explanatory! https://json5.org/

@Mechite Mechite changed the title JSON5 support [proposed Enhancement] JSON5 support Nov 9, 2022
@Mechite Mechite changed the title [proposed Enhancement] JSON5 support [proposed enhancement] JSON5 support Nov 9, 2022
@TheElectronWill
Copy link
Owner

interesting! thanks for the link

@Mechite Mechite changed the title [proposed enhancement] JSON5 support JSON5 support Nov 9, 2022
@IMB11
Copy link

IMB11 commented Nov 22, 2022

JSON5 is supported by the Jankson library.

@Mechite
Copy link
Author

Mechite commented Apr 23, 2023

Just a small comment (and bump) - is there actually much of a difference between JSON5 and HOCON, other than the other features HOCON has? - i.e. isn't all valid JSON5 also valid HOCON? If this is the case, I think we can simply point people to use NightConfig HOCON instead, though I don't know if the writing capabilities of the HOCON parser are going to be adequate for people who wish to use JSON5. JSON5 has many factors that make it attractive, perhaps the main one being that it is all valid JavaScript code, and the ease of enforcing convention on JSON5, where HOCON's ability to use = and omitting commas can be an issue.

@TheElectronWill TheElectronWill self-assigned this Jan 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants