Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#200 Allow a space in unquoted strings #202

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

fktn-k
Copy link
Owner

@fktn-k fktn-k commented Nov 8, 2023

As described here in the YAML specification, plain (unquoted) string scalars can contain a space character within the character sequence if a space character is surrounded by non-space characters, for instance:

test: This should be allowed but fkYAML throws an exception

fkYAML, however, did not support the exceptional case before this PR.
So, this PR has fixed the deserialization procedure and supported the above case when the precondition is met with the YAML specification.

@fktn-k fktn-k added the bug Something isn't working label Nov 8, 2023
@fktn-k fktn-k added this to the Release v0.2.1 milestone Nov 8, 2023
@fktn-k fktn-k self-assigned this Nov 8, 2023
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling d85d6e2 on feature/200_allow_space_in_unquoted_strings
into 7df5b59 on develop.

@fktn-k fktn-k merged commit 6ed3e07 into develop Nov 8, 2023
@fktn-k fktn-k deleted the feature/200_allow_space_in_unquoted_strings branch November 8, 2023 17:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants