-
-
Notifications
You must be signed in to change notification settings - Fork 417
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
Added parse_lark_grammar.py and the syntax parameter/registry #1019
base: master
Are you sure you want to change the base?
Conversation
It needs a little tidying up but overall it looks alright. A few things:
|
??
Fair enough. For some reason I thought it was more than just that. Currently the order is |
Oops sorry. I meant |
The last one. Unless you can think of a good reason to choose one of the others. |
My idea was to always fall back to |
That reminds me, we should also because of this start loading files with different extensions. |
That's a good point. I know some users had those for Lark 0.x I guess my thinking is that it's okay to force them to use If we do it your way, I have a few questions -
P.s. another option is that they can
And a general question - what happens if we have both |
I think the core design here stands. I would like feedback on that already.
Primarily missing here is tests, examples and docs.