-
Notifications
You must be signed in to change notification settings - Fork 556
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
Check for message names being null during validation #9164
Conversation
...ine/processing/deployment/model/validation/ProcessMessageStartEventMessageNameValidator.java
Outdated
Show resolved
Hide resolved
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.
@pihme looks good in general 👍
But I'm unsure about the rejection message. I have the feeling that we will mention the same validation twice. Please check my comment.
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.
@pihme LGTM 🚀
bors merge |
Successfully created backport PR #9192 for |
Successfully created backport PR #9193 for |
Description
Related issues
closes #9083
Definition of Done
Code changes:
backport stable/1.3
) to the PR, in case that fails you need to create backports manually.Testing:
Documentation: