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

Fix: conditional-flows multiple flows no conditions #131

Closed

Conversation

deanmzamzow
Copy link

Closes #130

@CLAassistant
Copy link

CLAassistant commented Feb 9, 2024

CLA assistant check
All committers have signed the CLA.

@deanmzamzow
Copy link
Author

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

@nikku
Copy link
Member

nikku commented Feb 15, 2024

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

@deanmzamzow
Copy link
Author

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.

@deanmzamzow deanmzamzow force-pushed the conditional-gateway-flows branch from 7d2faec to 1cb439f Compare February 15, 2024 14:59
@nikku
Copy link
Member

nikku commented Feb 15, 2024

Works now 👏

@nikku
Copy link
Member

nikku commented Feb 16, 2024

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.

@nikku nikku closed this Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

conditional-flows rule on multiple outgoing flows with no conditions
3 participants