Skip to content

Conversation

@montymxb
Copy link
Contributor

@montymxb montymxb commented Nov 6, 2025

Was looking into the bnf generator and realized it wasn't documented externally yet. Went ahead & added that plus an associated log entry to match the other generators.

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :)

@spoenemann
Copy link
Contributor

Oh, we have a schema doc? Could you add grammar validation options, too?

https://github.com/eclipse-langium/langium/pull/1951/files#diff-d77efe86f79f05410b1c0fd6aa7c16ad6543f1ee6f4397ec4ce2f735ed6db4eb

@montymxb
Copy link
Contributor Author

@spoenemann yep I can add those in today before pulling in the changes. No problem.

Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@montymxb montymxb merged commit 9da035c into main Nov 17, 2025
4 checks passed
@montymxb montymxb deleted the montymxb/document-bnf-gen branch November 17, 2025 13:16
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.

4 participants