Question: Should we allow duplicate keys in our JSON objects ?

[jbampton]

I was attempting to add the official pre-commit hook check-json

https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#check-json

And when testing the hook locally it showed two JSON files that had duplicate keys.

Should we standardize all our JSON files and enable checking or continue with the current inconsistent approach ?

So it just shows two but there are many in the "GR" file

There is a discussion below about this:

https://stackoverflow.com/questions/21832701/does-json-syntax-allow-duplicate-keys-in-an-object

[DaanHoogland]

In case you would need an array of values , having duplicate keys would make sense, I do not think we would have syuch a use-case. I think we can study the English and Greek locales and remove the duplicates. My English is ok-ish, but my Greek is both ancient and not of any level to speak of.

I think it makes sense to install this check.

[shwstppr]

I think check-json can be added. Duplicate keys are sometimes added by mistake, especially when resolving merge conflicts.