fktn-k / fkYAML

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

Fix false error on anchor names containing colons (:) #335

Closed fktn-k closed 5 months ago

fktn-k commented 5 months ago

The latest lexer emits a wrong error on anchor names which contains colons(:).
According to the YAML spec, anchor names consist of printable characters (including colons) EXCEPT newline codes, white spaces and flow indicators.
So, this PR has corrected the way of extracting an anchor name from the input source to follow the spec by modifying the lexer and the related test cases.


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

Name fkYAML_coverage.pr335.zip
ID 1467008814
URL https://github.com/fktn-k/fkYAML/actions/runs/8923800240/artifacts/1467008814
coveralls commented 5 months ago

Pull Request Test Coverage Report for Build 8923800240

Details


Totals Coverage Status
Change from base Build 8920856596: 0.0%
Covered Lines: 3570
Relevant Lines: 3570

💛 - Coveralls