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

Support bpmn-js@7 in bpmn-js Extensions #1837

Closed
6 tasks done
philippfromme opened this issue Jun 16, 2020 · 5 comments · Fixed by #1864
Closed
6 tasks done

Support bpmn-js@7 in bpmn-js Extensions #1837

philippfromme opened this issue Jun 16, 2020 · 5 comments · Fixed by #1864

Comments

@philippfromme
Copy link
Contributor

philippfromme commented Jun 16, 2020

What should we do?

Update dev and peer dependencies and ensure extensions work with bpmn-js@7:

It might make sense to create issues in the respective repositories.

@linus-amg
Copy link

Out of curiosity, will you just add 7 to the list, or already plan ahead and add 8 and such? Or will this always be a topic to do after releasing a major?

@philippfromme
Copy link
Contributor Author

We do not add versions that have been released yet because we can not be sure whether they will be compatible without changes. Of course we should always update peer dependencies as quickly as possible after a major release.

@barmac
Copy link
Contributor

barmac commented Jun 19, 2020

I think that until we know what breaking changes are introduced in bpmn-js@8, we cannot guarantee that the package will work with that major version.

@linus-amg
Copy link

We do not add versions that have been released yet because we can not be sure whether they will be compatible without changes. Of course we should always update peer dependencies as quickly as possible after a major release.

Oh well yeah, that obviously makes a lot of sense 🥟 thanks for the fast reply!

@pinussilvestrus
Copy link
Contributor

Closed via 6780501

@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jul 10, 2020
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 a pull request may close this issue.

4 participants