Maciej Barelkowski
Maciej Barelkowski
OK it works (ignore the import secrets failure). We will merge this within a couple of days as we are now in the code freeze. Thank you for your PR!
I just squashed and merged this contribution. Thank you @AlexanderSkrock!
This will be implemented upstream via https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/111
Fixed upstream via https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/111
As a consequence of the decision PHub epic, I am reopening this issue, moving to the backlog, and removing it from the 8.3 milestone.
Fixed upstream via https://github.com/camunda/bpmnlint-plugin-camunda-compat/pull/166
In the improved-canvas library, we [register `appendCreatePad` action](https://github.com/camunda/improved-canvas/blob/9c8a56835886772d3bf9320b0ec186ae7d39a5e5/lib/bpmn/appendCreatePad/AppendEditorActions.js#L40), and [deregister `appendElement`](https://github.com/camunda/improved-canvas/blob/9c8a56835886772d3bf9320b0ec186ae7d39a5e5/lib/bpmn/appendCreatePad/AppendEditorActions.js#L20). Somehow the latter action is still kept for the keyboard binding though.
We already have it in the name field of the BPMN editor.
This will be fixed via https://github.com/bpmn-io/bpmn-js/pull/2194
Ideas: 1. Run linter in a worker (difficult because we need to pass the model) 2. Limit the number of elements tested for the no-loop rule The rule is implemented...