Skip to content

Conversation

@Buckwich
Copy link
Member

@Buckwich Buckwich commented Aug 20, 2025

Proposed Changes

bpmn-io/bpmn-moddle#131 changed (ie removed) the default: true for cancelRemainingInstances. This change makes the default (isEdited) detection reflect that change

Checklist

To ensure you provided everything we need to look at your PR:

  • Brief textual description of the changes present
  • Visual demo attached
  • Steps to try out present, i.e. using the @bpmn-io/sr tool
  • Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Aug 20, 2025
@Buckwich Buckwich force-pushed the fix-cancelremaintinstances-default branch from 499b763 to e5914ff Compare August 20, 2025 13:05
@Buckwich Buckwich marked this pull request as ready for review August 20, 2025 13:15
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Aug 20, 2025
@Buckwich Buckwich requested a review from philippfromme August 20, 2025 13:15
@philippfromme philippfromme force-pushed the fix-cancelremaintinstances-default branch from e5914ff to 2660b92 Compare August 20, 2025 13:18
Change in bpmn-moddle requires latest bpmn-js
Copy link
Contributor

@philippfromme philippfromme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed the tests and also bumped the bpmn-js peer dependency to the latest which is required due to the change in bpmn-moddle.

@philippfromme philippfromme merged commit d7ebc1d into main Aug 20, 2025
9 checks passed
@philippfromme philippfromme deleted the fix-cancelremaintinstances-default branch August 20, 2025 13:40
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Aug 20, 2025
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.

3 participants