-
Notifications
You must be signed in to change notification settings - Fork 0
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
Configurable language parsing #23
base: main
Are you sure you want to change the base?
Conversation
This enables us to pass further context into the parser.
I'd be interested in your feedback @barmac and @marstamm: What representation shall we use for |
I think that depends on how we want to use it. We use it in |
Some use cases I can see for this tool in DMN editor are:
To support this, the context should not assume that we can pass exact values. |
Thanks for your feedback folks. What I get is the following:
|
I am making it a draft as it's still in progress. |
Proposed Changes
This allows us to configure custom language parsing to be used. This way we can validate (using text validation)
expression
andunaryTests
, but also provide custom context.Related to camunda/camunda-modeler#3983
Checklist
To ensure you provided everything we need to look at your PR:
@bpmn-io/sr
toolCloses {LINK_TO_ISSUE}
orRelated to {LINK_TO_ISSUE}