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

feat: support engines property #152

Merged
merged 2 commits into from
Dec 6, 2024
Merged

feat: support engines property #152

merged 2 commits into from
Dec 6, 2024

Conversation

jarekdanielak
Copy link
Contributor

Closes #146

Add engines property to include a list of platforms and their versions compatible with element template.

@jarekdanielak
Copy link
Contributor Author

jarekdanielak commented Dec 6, 2024

Since we agreed not to have any additional validation for semver here, it's ready for review.

Question: Do we agree on camunda being a standard and suggested name for our platform or do we go with something else: camundaCloud, zeebe etc.?

It should be camunda. I don't think we should explicitly validate it as part of this PR though (could be anything in other contexts).

@jarekdanielak jarekdanielak marked this pull request as ready for review December 6, 2024 08:06
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Dec 6, 2024
@jarekdanielak jarekdanielak requested a review from barmac December 6, 2024 08:30
jarekdanielak and others added 2 commits December 6, 2024 10:44
`engines` is a new property that allows templates to declare
compatibility with other run-time provided engines.

Closes #146
Copy link
Member

@nikku nikku left a comment

Choose a reason for hiding this comment

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

Reviewed, including a manual test.

Added additional test for "no camunda provided", improved descriptions, and fixed validationSpec to prevent undetected use of it.only in the future 🏁

@nikku nikku merged commit 32b05ef into main Dec 6, 2024
3 checks passed
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Dec 6, 2024
@nikku nikku deleted the engines branch December 6, 2024 09:46
nikku added a commit to bpmn-io/bpmn-js-element-templates that referenced this pull request Dec 11, 2024
nikku added a commit to bpmn-io/bpmn-js-element-templates that referenced this pull request Dec 12, 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.

Add property for supported runtime versions
2 participants