yaml / yaml-test-suite

Comprehensive, language independent Test Suite for YAML
MIT License
172 stars 58 forks source link

Fix WZ62 out.yaml #28

Closed eemeli closed 6 years ago

eemeli commented 6 years ago

The current production has !!str: bar, which does not match the intent -- : is a valid ns-tag-char, so that gets parsed as the value bar with a tag!!str:.

To fix, a space should be added before the :.

perlpunk commented 6 years ago

Thanks! We are also working on other fixes and have done some in a devel branch. We're all pretty busy right now, I hope I can have a look at this and your other PRs at the weekend. Feel free to join us on IRC freenode#yaml-dev

perlpunk commented 6 years ago

I'll merge this because the out.yaml is currently broken.

We haven't yet decided how we are going to use out.yaml. We were discussing to add

For both we actually need to define some kind of canonical output.

By adding emit.yaml a processor can test its emitter in an isolated way without loading the data structure.

As you can see, this is all still work in progress, especially the emitter part.

perlpunk commented 6 years ago

merged, thanks!