-
Notifications
You must be signed in to change notification settings - Fork 7
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
KDL 2.0 compliance #21
Comments
Chevrotain v11 (the parser lib) is ESM-only now, how do you feel about moving to ESM (or async |
There don't seem to be a great deal of other changes since v10 so I'd be fine with keeping that for now. https://chevrotain.io/docs/changes/CHANGELOG.html |
I am, in general, in favor of ESM-ification, tbh. But if there's no big changes worth pulling in, compat is probably a better idea. |
Currently considering whether to just test for the DLCPs before parsing, or including them in the lexer (and complicating the token patterns in the process). |
KDL 2.0.0 is now released. :-) It would be great if this library got an update because then we could update the playground. (https://kdl-play.danini.dev/) |
@dezren39 working on it now (#24). Note that https://kdl-play.danini.dev/ uses a different JS library for KDL, namely https://github.com/bgotink/kdl |
@dezren39 @larsgw there’s an “official” playground now at https://kdl.dev/play that runs KDL v2 |
I am not sure what the appeal of this library over https://github.com/bgotink/kdl would be anymore. |
@zkat okay, it supports the tests now, and I've converted it to ESM. |
@larsgw idk. It's simpler, perhaps lighter/faster? And also in general I think it's good to have more options tbh. |
KDL 2.0.0-draft.3 is now generally available (kdl-org/kdl#286), and with it, we've decided to encourage existing implementations to start working towards 2.0 support and send feedback and suggestions for anything you might run into, so I'm making this tracking issue. :)
The text was updated successfully, but these errors were encountered: