-
Notifications
You must be signed in to change notification settings - Fork 66
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
getAllReachableRules should return implicit ID rule #1151
Labels
bug
Something isn't working
Comments
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 15, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 15, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 15, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 15, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 15, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 16, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 16, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 16, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
cdietrich
added a commit
to cdietrich/langium
that referenced
this issue
Aug 16, 2023
Fixes eclipse-langium#1151 Signed-off-by: Christian Dietrich <christian.dietrich.opensource@gmail.com>
As mentioned in #1152, there is no magic/implicit usage of the ID rule. In the given example, Langium uses the STRING rule because that's what is assigned to the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
getAllReachableRules
should return implicitID
rule but currently does not.this leads to error initializing a parser based on a imported grammar
that only uses rules that call ID implicitely
grammar-util.test.ts
The text was updated successfully, but these errors were encountered: