search

json-ld / yaml-ld

CG specification for YAML-LD and UCR
https://json-ld.github.io/yaml-ld/spec
Other
19 stars 8 forks source link

Description of converting YAML to the internal representation. #65

Closed gkellogg closed 1 year ago

gkellogg commented 1 year ago

Fixes #62.

Preview | Diff

gkellogg commented 1 year ago

This should be complete now. I'm sure it's inadequate in its treatment of tags. I think the named anchors and anchor nodes are handled consistently with the YAML spec, though.

This needs a careful review from those more familiar with the details of YAML.

ioggstream commented 1 year ago

@gkellogg did you get some feedback from YAML folks on this topic?

gkellogg commented 1 year ago

@gkellogg did you get some feedback from YAML folks on this topic?

No, not yet.

One necessary improvement to my text is to provide some handling of looping alias nodes along the lines of "when resolving aliases implementations MUST check for recursive entry of a referenced node and fail."

As I said, I think a process such as this should be defined elsewhere and referenced, but we'll keep it here until something better comes along. Improvements and sanity check, of course, are always welcome.

gkellogg commented 1 year ago

Note the comment from b95391285a5eee8f94d0a495921bb5ff347e48a3:

This may be largely a matter of opinion, for which there is no right answer, as writing algorithmic text is not quite the same as normal English.

Still, we strive for perfection in all things...

gkellogg commented 1 year ago

This issue was discussed on the Aug 03 meeting. and we agreed to merge.