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.

Sign up for GitHub

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

Jump to bottom

Record syntax error instead of panicing when no block found #109

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Record syntax error instead of panicing when no block found #109

merged 1 commit into from
Feb 6, 2024

Commits on Feb 6, 2024

  1. Record syntax error instead of panicing when no block found 

    Before, if we expect a block, beginning with `{`, the parser panics.
Now it logs an appropriate syntax error.

It could use better recovery. Because later errors are logged which
essentially say the same thing. But analyzing the cases for recovery
my take a bit of logic.

Closes #107
    @jlapeyre
    jlapeyre committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    3c69b71 View commit details
    Browse the repository at this point in the history