-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Description
Currently all objects in the SDK are mutable. If a new event or object is needed, it should be recreated, instead of using setters to set things.
sdk-java/sdk/src/main/java/dev/cdevents/models/build/started/Subject.java
Lines 113 to 115 in 5d17b75
| public void setContent(Content content) { | |
| this.content = content; | |
| } |
Instead we should utilize the builder pattern, imo. Look at lombok, https://projectlombok.org/features/Builder. This will make it a lot easier to generate these and make them properly immutable
Metadata
Metadata
Assignees
Labels
No labels