search

cdevents / sdk-java

Java SDK for CDEvents
Apache License 2.0
5 stars 6 forks source link

Updating the Java-SDK to the CDEvents Spec - v0.1.0 #12

Closed adamkenihan closed 1 year ago

adamkenihan commented 1 year ago

Updated events to align with version 0.1.0 spec as described with issue #11

afrittoli commented 1 year ago

Thanks, @adamkenihan for this!

How do you plan on managing the migration to the new format? If we merge this to main it might be worth adding a note to the README about only using a released version of this and not main - since the SDK would be in a mixed state until the migration is complete.

e-backmark-ericsson commented 1 year ago

@rjalander , will you be able to review this so it can be merged?

rjalander commented 1 year ago

@adamkenihan please update the PR description as the changes are not only for pipeline events and mention the changes along with the issue https://github.com/cdevents/sdk-java/issues/11

rjalander commented 1 year ago

Thank you @adamkenihan Reviewed and the changes looks good to me.

rjalander commented 1 year ago

Closing this PR as the same changes raised through Nordix repo https://github.com/Nordix/sdk-java with PR https://github.com/cdevents/sdk-java/pull/14