search

bpmn-io / bpmnlint

Validate BPMN diagrams based on configurable lint rules.
MIT License
119 stars 36 forks source link

Fix: `conditional-flows` multiple flows no conditions #131

Closed deanmzamzow closed 5 months ago

deanmzamzow commented 5 months ago

Closes #130

CLAassistant commented 5 months ago

CLA assistant check
All committers have signed the CLA.

deanmzamzow commented 5 months ago

I've signed the CLA. I don't know why it's not updating in the PR check. image

nikku commented 5 months ago

@deanmzamzow I'm not sure either. Did you login via different accounts or use a different email to commit these changes?

deanmzamzow commented 5 months ago

Yes, I created the commit with a different (work-only) github account, and then created this account to actually submit the PR.

I'll rebase so that the commit is created with this account instead.

nikku commented 5 months ago

Works now :clap:

nikku commented 5 months ago

Closed the linked issue as won't fix (cf. rationale). Strict "modeling for execution" rules are maintained in separate places, i.e. for Camunda 8 it would be bpmnlint-plugin-camunda-compat.