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

Fix: support for compiled jsons ending with an container and not a DONE #56

Merged
merged 2 commits into from
Sep 22, 2023
Merged

Conversation

JBenda
Copy link
Owner

@JBenda JBenda commented Sep 22, 2023

As shown by Jerry
adds inky, an empty container at the end of the story.
Which resulted in an unexpected EOF.

Now there is a check iff we leave the top-level container and are
at the end of the story file, the story ends.

Ensures compatibility for different ink compilers
(bonus: use link for inkcpp_test resource file
should now also support Windows)

as shown by [Jerry](https://discordapp.com/users/jerrytron)
adds inky a empty container at the end of the story.
Which resultet in unexpected EOF.

Now there is a check iff we leave the top level container and are
at the end of the story file, the story ends.
Ensures compability for different ink compilers
(bonus: use link for inkcpp_test resource file
should now also supported on windows)
@JBenda JBenda merged commit 59a774c into JBenda:master Sep 22, 2023
5 checks passed
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.

1 participant