fktn-k / fkYAML

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

Fix parse error on empty flow containers #351

Closed fktn-k closed 4 months ago

fktn-k commented 4 months ago

As reported in the issue #347, the latest parser mistakenly emits the parse errors on empty flow containers like:

foo: {}
bar: []

They are caused because the PR #350 lacked implementation for empty flow containers.
So, this PR has fixed them by adding the missing implementation 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 4 months ago

:octocat: Upload Coverage Event Notification

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

Name fkYAML_coverage.pr351.zip
ID 1540759864
URL https://github.com/fktn-k/fkYAML/actions/runs/9254185815/artifacts/1540759864
coveralls commented 4 months ago

Pull Request Test Coverage Report for Build 9254185815

Details


Totals Coverage Status
Change from base Build 9247789643: 0.0%
Covered Lines: 3897
Relevant Lines: 3897

💛 - Coveralls