Fix #15 by adding . (dot) between event type name and version if missing - #24

@afrittoli seems like the /approve trick didn't work, this PR still requires a review.

@zaza heh, we don't have prow installed, silly me - if you rebase the PR I will merge it then

Thanks for the update. Could you please remove the merge commit from the PR?

I'm sorry, I thought you're able to squash and merge the PR.

I'm sorry, I thought you're able to squash and merge the PR.

No worries, probably the squash will remove the merge commit, let me try :)

@zaza @afrittoli, Currently refactoring the code the way we need to create a CDEvent as per the latest spec.
and here we used CDEvents Spec Version instead of event version as per the comment #35 (comment)

As per my knowledge will not be using https://github.com/cdevents/sdk-java/blob/main/src/main/java/dev/cdevents/CDEventTypes.java, once this PR is available with all other events #35

@zaza @afrittoli, Currently refactoring the code the way we need to create a CDEvent as per the latest spec. and here we used CDEvents Spec Version instead of event version as per the comment #35 (comment)

As per my knowledge will not be using https://github.com/cdevents/sdk-java/blob/main/src/main/java/dev/cdevents/CDEventTypes.java, once this PR is available with all other events #35

Thanks, @zaza for this PR and @rjalander for your comment.
It sounds like we should be closing this one then - @zaza your contributions are very welcome nonetheless!
I assigned issues that are covered by #35 to @rjalander, to avoid double work on any of those.

@zaza in case you'd like to chat about the SDK and CDEvents, we have a #cdevents channel on the CDF slack, a mailing list and weekly working groups.

Closing the PR as requested.