-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document that YAML::Tiny is not YAML #57
Comments
Isn't it the other way around, that YAML::Tiny only reads and produces a limited subset of the YAML dialect, so if you use some of the more advanced features, YAML::Tiny cannot read it? I don't think it's intentional that YAML::Tiny produces non-compliant output. We should fix that! (I am definitely in favour of updating the documentation to make clear what YAML::Tiny is good for, and not good for.) |
I believe the issue is that it's not truly a subset of YAML. It mostly is, but there are things that it just parses differently from how a real YAML parser would. |
Yeah, sorry =) It's that it accepts things that are not valid. The problem is that if people are using YAML::Tiny in their software, they are likely to write invalid YAML at some point. The question is, could we fix all these cases? Maybe we can, but it would break things. We recently had the case at work.
And then the include happened via post processing. In our case it was easy to fix because this feature was so far only used locally, so we were able to fix all YAML files, and we used I can try to look into the issues and see if those are easily fixable, but like I said, it might then break things. |
While YAML::Tiny is great and fast for really simple data, I think the documentation should be clearer that it is actually not YAML.
It says it's a subset, but that makes people think that files that can be read by YAML::Tiny can also be read by other YAML libs.
But that's not the case. It can lead to syntax that is not valid YAML.
That creates the problem that people using it cannot switch to a different module easily. If all YAML files are controlled by the same person, that might work, but otherwise it will break things.
The text was updated successfully, but these errors were encountered: