contest icon indicating copy to clipboard operation
contest copied to clipboard

[v2] make job descriptor JSON5 compatible

Open mimir-d opened this issue 3 years ago • 3 comments

Current state: the serialization/deserialization of JSON is done with the golang "encoding/json" package which only understands the standard ECMA flavor. It has proven useful to have comments in the input descriptors, so we should use a format that supports that.

mimir-d avatar Aug 22 '22 15:08 mimir-d

Current state: the serialization/deserialization of JSON is done with the golang "encoding/json" package which only understands the standard ECMA flavor.

IIRC, we had a consensus (in old discussions with @insomniacslk) to use YAML. It is human-readable, supports comments and is a superset for JSON.

xaionaro avatar Aug 22 '22 17:08 xaionaro

well, we have both. And we can't just remove support for json now. This would at least get some parity.

mimir-d avatar Aug 23 '22 14:08 mimir-d

We do not need the support of JSON, since it is a subset of YAML.

xaionaro avatar Aug 24 '22 13:08 xaionaro