The very first moment that I had to use JSON as a configuration format, and I was desperate to find a way to make a long string into a JSON field. JSON is great for many things, but it’s not good at all for a configuration format where you need users to make it pretty, and need features like comments or multi-line strings (because you don’t want to fix a merge conflict in a 400 character-wide line).
Comment on Everything about TOML format - Orchard Dweller
brettvitaz@programming.dev 9 months agoSo it’s good for things that don’t change, or things you have to change by hand, which gives it almost zero utility for any project that I’ve worked on.
suy@programming.dev 9 months ago
spartanatreyu@programming.dev 9 months ago
Where do you put your comments in JSON files?
Quetzalcutlass@lemmy.world 9 months ago
I’ve seen them included as part of the data.
“//”, “Comment goes here”,
spartanatreyu@programming.dev 9 months ago
That doesn’t really work when you need two comments at the same level, since they’d both have the same key
vrighter@discuss.tchncs.de 9 months ago
write json with comments. Use a yaml parser.
catfish@lemmy.ml 9 months ago
It still works since multiple identical keys are still valid json. Although that in itself isn’t fantastic imo.
brettvitaz@programming.dev 9 months ago
For settings files I always have an example file with sensible values filled in and along with descriptive keys that serves as reasonable documentation. If something is truly unknowable, I’ve probably done something wrong.
spartanatreyu@programming.dev 9 months ago
How would you mark a flag in your json settings file as deprecated?
brettvitaz@programming.dev 9 months ago
In my opinion, the settings file isn’t where this information should be presented. I would put these notes in the release log and readme and example settings file. I have also written this information to logging during startup so a user knows what to do, or I write a migration that does the change automatically if that’s possible.
This is only my opinion and you can use the comment method described like
“//“: “Deprecated”
if desired.