-
Notifications
You must be signed in to change notification settings - Fork 565
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
Do not generate ImmutableProcessInstanceRelated #9014
Conversation
Do not generate `ImmutableProcessInstanceRelated` as it is not a leaf type, and does not need an immutable variant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💯
bors merge |
9014: Do not generate ImmutableProcessInstanceRelated r=npepinpe a=npepinpe ## Description `ProcessInstanceRelated` was mistakenly annotated with the immutable annotation, which caused us to generate an unnecessary class. This simply removes it, and adds a regression test. Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
This PR was included in a batch that successfully built, but then failed to merge into main (it was a non-fast-forward update). It will be automatically retried. |
9014: Do not generate ImmutableProcessInstanceRelated r=npepinpe a=npepinpe ## Description `ProcessInstanceRelated` was mistakenly annotated with the immutable annotation, which caused us to generate an unnecessary class. This simply removes it, and adds a regression test. Co-authored-by: Nicolas Pepin-Perreault <nicolas.pepin-perreault@camunda.com>
Build failed: |
bors merge |
Sorry about that, my fault |
Successfully created backport PR #9028 for |
Description
ProcessInstanceRelated
was mistakenly annotated with the immutable annotation, which caused us to generate an unnecessary class. This simply removes it, and adds a regression test.Definition of Done
Not all items need to be done depending on the issue and the pull request.
Code changes:
backport stable/1.3
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation: