Skip to content
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

Include no-bpmndi rule #2

Open
MaxTru opened this issue Jun 7, 2022 · 1 comment
Open

Include no-bpmndi rule #2

MaxTru opened this issue Jun 7, 2022 · 1 comment
Labels
backlog Queued in backlog rules

Comments

@MaxTru
Copy link

MaxTru commented Jun 7, 2022

Problem

We already had multiple cases where deployments to Camunda failed or went to unexpected results, because of missing bpmndi info.

See:

Solution proposal

The no-bpmndi rule detects such cases: https://github.com/bpmn-io/bpmnlint/blob/master/rules/no-bpmndi.js. We could easily include it in our standard camunda-compat ruleset. It also fits well, since it ensures, that the diagram can successfully be deployed to Camunda.

Risk

Risk: the no-bpmndi rule traverses the entire model, hence it might have performance implications on large diagrams. => to be tested prior to executing the proposal.

@philippfromme
Copy link
Collaborator

@christian-konrad Has it already been decided that we will include this rule with @camunda/linting? Or is there more discussion needed?

@nikku nikku added the rules label Sep 27, 2022
@philippfromme philippfromme removed their assignment Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog rules
Projects
None yet
Development

No branches or pull requests

4 participants