-
Notifications
You must be signed in to change notification settings - Fork 65
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
Made grammar object optional #381
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good at first glance. I'll take a closer look at it later.
On another note, do you mind running the langium generator on the arithmetics and statemachine example as well? Their grammar.ts
should be updated before merging this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a validation check to langium-grammar-validator.ts
: A rule marked as entry
is illegal if the containing grammar is not declared.
771ef75
to
a2a0417
Compare
Co-authored-by: Mark Sujew <mark.sujew@typefox.io>
Grammar top level object is optional now.
However if it is missing in grammar which is configured in langium-config.json then langium generator fails.
Fixes #299