Skip to content

Grammar validation #311

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

Merged
merged 3 commits into from
May 4, 2021
Merged

Conversation

BillWagner
Copy link
Member

@RexJaeschke

I found the remaining syntax issues while reviewing #261

Since most of these weren't in placed you'd changed, I made a fresh PR to address those.

Once you accept these, the only remaining validation issue is the mutual left recursion issue.

RexJaeschke and others added 3 commits April 15, 2021 09:42
* Add lexer rules to grammar validation

Include the stub definitions for lexer members into the grammar file.

* Fix a script error. > vs. >>
After these changes, the only concerns are the mutual left recursion rules.
@BillWagner BillWagner requested a review from RexJaeschke May 4, 2021 20:52
@RexJaeschke RexJaeschke merged commit 0cd4ecb into dotnet:draft-v6 May 4, 2021
@BillWagner BillWagner deleted the grammar-validation branch May 5, 2021 13:27
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.

2 participants