fktn-k / fkYAML

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

Fix parse error on the first node which isn't a block mapping #322

Closed fktn-k closed 5 months ago

fktn-k commented 5 months ago

fkYAML in the latest develop branch emits a parse error when an input buffer starts with a node which is not a block mapping, like the following valid YAML snippets:
(block sequence)

- 123
- foo
- 3.14

(flow sequence)

[123, foo, 3.14]

(flow mapping)

{foo: 123, true: 3.14}

So, this PR has fixed the error 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 5 months ago

:octocat: Upload Coverage Event Notification

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

Name fkYAML_coverage.pr322.zip
ID 1450948658
URL https://github.com/fktn-k/fkYAML/actions/runs/8849043964/artifacts/1450948658
coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 8849043964

Details


Totals Coverage Status
Change from base Build 8835266680: 0.0%
Covered Lines: 3544
Relevant Lines: 3544

💛 - Coveralls