fktn-k / fkYAML

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

Detect errors on missing flow value separators #350

Closed fktn-k closed 4 months ago

fktn-k commented 4 months ago

This PR has added error detection of missing flow value separators ( , ) in flow containers and some test cases for the detection.
Before this PR, the deserializer mistakenly succeeded in parsing invalid YAML snippets which lacks necessary value separators, like the following:

# There must be a value separator between `123` and `true`.
foo: [123  true, 3.14]

# There must be a value separator between `123` and `qux`.
bar: {baz: 123  qux: false}

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 2d7356c95682e8e7d794203b41efdee2f4f9baf2. You can download the artifact which contains the same file uploaded to the Coveralls and its HTML version.

Name fkYAML_coverage.pr350.zip
ID 1539333621
URL https://github.com/fktn-k/fkYAML/actions/runs/9247643918/artifacts/1539333621