You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We already integrate Yojson, so we might as well use it for everything. Currently the config system uses our own JSON structures and we have our own JSON lexer and parser.
There's also JsonSchema that checks this custom JSON structure, but if we want to keep that, I guess it could still be adapted to just look at a Yojson structure instead.
The text was updated successfully, but these errors were encountered:
So when setting options, we just mutate some ref inside it and that's it. To use Yojson, one would have to recursively reconstruct the modified structure around the change.
We already integrate Yojson, so we might as well use it for everything. Currently the config system uses our own JSON structures and we have our own JSON lexer and parser.
There's also
JsonSchema
that checks this custom JSON structure, but if we want to keep that, I guess it could still be adapted to just look at a Yojson structure instead.The text was updated successfully, but these errors were encountered: