night-config icon indicating copy to clipboard operation
night-config copied to clipboard

JSON5 support

Open Mechite opened this issue 1 year ago • 3 comments

Self explanatory! https://json5.org/

Mechite avatar Nov 09 '22 16:11 Mechite

interesting! thanks for the link

TheElectronWill avatar Nov 09 '22 16:11 TheElectronWill

JSON5 is supported by the Jankson library.

IMB11 avatar Nov 22 '22 22:11 IMB11

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.

Mechite avatar Apr 23 '23 16:04 Mechite