Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Valid example file was invalid! #144

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

DimitriPapadopoulos
Copy link
Contributor

Fix valid-0013.json added by fab533e (#125)

@LB--
Copy link
Member

LB-- commented Oct 30, 2021

See discussion in #73 - it may not be valid according to the JSON specification, but the file is named as such because it is intended to be accepted by this library.

EDIT: Perhaps it should have ext in the filename though...

@DimitriPapadopoulos
Copy link
Contributor Author

That's the file I added recently. I didn't intend it to have the comma, it should just check Unicode support, not extensions.

@LB--
Copy link
Member

LB-- commented Nov 2, 2021

Ah I see, I forgot the file was introduced by you to begin with! Thanks for reminding me.

@LB-- LB-- added this to the v1.1.1 milestone Nov 2, 2021
@LB-- LB-- merged commit b9b5d53 into json-parser:master Nov 2, 2021
@DimitriPapadopoulos DimitriPapadopoulos deleted the fix_fab533e branch November 3, 2021 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants