search

kdl-org / kdl

the kdl document language specifications
https://kdl.dev
Other
1.1k stars 61 forks source link

Start draft of KDL Data Model #228

Open nichtich opened 2 years ago

nichtich commented 2 years ago

Thanks for discussion (https://github.com/kdl-org/kdl/issues/225). In addition to further changes to the specification of KDL Data model, the document should be referenced from other documents in this repository.

tabatkins commented 2 years ago

+1, these changes are nice. I can update the tests to mandate empty tags not be preserved in serialization once this is merged.

zkat commented 2 years ago

moving this to kdl-v2

nichtich commented 2 years ago

moving this to kdl-v2

I don't think the data model breaks the KDL spec because KDL spec does not define how to internally store the result of parsing. A model is more important for KQL and KDL Schema. Nevertheless it would help to state that a KDL parser MUST preserve at least the information covered by the data model.

tabatkins commented 2 years ago

Agreed, this should just be capturing the assumptions inherent in the test-suite, and so is suitable for v1.

zkat commented 2 years ago

Noted. Moved back :)

zkat commented 2 years ago

Shall we roll this up as part of KDL 2.0?