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

Crawl Directives compile error messages are awful #3

Open
tokahuke opened this issue Aug 26, 2020 · 1 comment
Open

Crawl Directives compile error messages are awful #3

tokahuke opened this issue Aug 26, 2020 · 1 comment
Labels
Lopez Crawl Directives This issue has to do with Crawl Directives language definition, compilation, etc...

Comments

@tokahuke
Copy link
Owner

tokahuke commented Aug 26, 2020 •
edited
Loading

The current compiler now

  1. Just shows the first error it finds.
  2. Has only the nom default error messages to fall back to to. And those messages are all but intuitive. Even I have a hard time getting the problem in .lcd code and I designed the syntax!

We need more insightful error messages.
@tokahuke tokahuke added good first issue Good for newcomers Lopez Crawl Directives This issue has to do with Crawl Directives language definition, compilation, etc... labels Aug 26, 2020
@tokahuke tokahuke changed the title Crawl Directives compile error messages are aweful Crawl Directives compile error messages are awful Aug 30, 2020
@tokahuke tokahuke removed the good first issue Good for newcomers label Sep 2, 2020
@tokahuke
Copy link
Owner Author

tokahuke commented Sep 4, 2020

On v0.4.0, it at least points to the line and column of the error (and there is a bit of ANSI term fluff thrown into).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Lopez Crawl Directives This issue has to do with Crawl Directives language definition, compilation, etc...
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tokahuke