fktn-k / fkYAML

A C++ header-only YAML library
MIT License
67 stars 7 forks source link

Apply node properties for the root node #370

Closed fktn-k closed 1 month ago

fktn-k commented 1 month ago

fkYAML currently ignores node properties for the root node unintentionally, like parsing the following valid YAML snippet:

--- !!map
foo: bar

They are obviously allowed in the spec.
So this PR has fixed the serializer/deserializer to properly handle them and added some test cases to validate the changes.


Pull Request Checklist

Read the CONTRIBUTING.md file for detailed information.

Please don't

github-actions[bot] commented 1 month ago

:octocat: Upload Coverage Event Notification

Coverage data has been uploaded for the commit e934d700df7a612226900ce63b874fb02b2dc887. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr370.zip
ID 1823046897
URL https://github.com/fktn-k/fkYAML/actions/runs/10432953001/artifacts/1823046897
coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 10432953001

Details


Totals Coverage Status
Change from base Build 10429308281: 0.0%
Covered Lines: 3971
Relevant Lines: 3971

💛 - Coveralls