-
Notifications
You must be signed in to change notification settings - Fork 6
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
Full Continuous Deployment CDEvents to render as CloudEvent and schema validation #50
Full Continuous Deployment CDEvents to render as CloudEvent and schema validation #50
Conversation
…vents' into mergingtest # Conflicts: # pom.xml # src/main/java/dev/cdevents/constants/CDEventConstants.java # src/test/java/dev/cdevents/CDEventsTest.java
* @param subjectType | ||
* sets the subject type | ||
*/ | ||
public void setSubjectType(CDEventConstants.SubjectType subjectType) { |
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.
Subject Type will be set to "service" for this event, method to set the type is not needed.
* @param subjectType | ||
* sets the subject type | ||
*/ | ||
public void setSubjectType(CDEventConstants.SubjectType subjectType) { |
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.
same as above comment, can be removed.
* @param subjectType | ||
* sets the subject type | ||
*/ | ||
public void setSubjectType(CDEventConstants.SubjectType subjectType) { |
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.
same as above comment, can be removed.
* @param subjectType | ||
* sets the subject type | ||
*/ | ||
public void setSubjectType(CDEventConstants.SubjectType subjectType) { |
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.
same as above comment
* @param subjectType | ||
* sets the subject type | ||
*/ | ||
public void setSubjectType(CDEventConstants.SubjectType subjectType) { |
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.
same as above comment
… some lint errors.
🎉 This issue has been resolved in |
This PR is a continuation of #44, to implement other Continuous Deployment events