Skip to content

Add clarity text and an example of versioning eventTypes#340

Merged
duglin merged 2 commits intocloudevents:masterfrom
duglin:versionEventType
Nov 29, 2018
Merged

Add clarity text and an example of versioning eventTypes#340
duglin merged 2 commits intocloudevents:masterfrom
duglin:versionEventType

Conversation

@duglin
Copy link
Copy Markdown
Collaborator

@duglin duglin commented Nov 3, 2018

Just because @ac360 wanted to see it.

Signed-off-by: Doug Davis dug@us.ibm.com

Comment thread spec.md Outdated
the organization which defines the semantics of this event type.
* Examples
* com.github.pull.create
* com.example.object.v2.delete
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

com.example.object.delete.v2

Just because Austen wanted to see it.

Signed-off-by: Doug Davis <dug@us.ibm.com>
Signed-off-by: Doug Davis <dug@us.ibm.com>
@duglin
Copy link
Copy Markdown
Collaborator Author

duglin commented Nov 12, 2018

@clemensv see if the text I added is headed in the direction you were thinking of on the call the other day. Any suggested edits are welcome - from anyone.

@duglin
Copy link
Copy Markdown
Collaborator Author

duglin commented Nov 21, 2018

@clemensv any comment on the latest edits?

@clemensv
Copy link
Copy Markdown
Contributor

I like the guidance part. LGTM.

Copy link
Copy Markdown

@Vlaaaaaaad Vlaaaaaaad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@duglin
Copy link
Copy Markdown
Collaborator Author

duglin commented Nov 29, 2018

Approved on 11/29 call

@duglin duglin merged commit 75c8118 into cloudevents:master Nov 29, 2018
@duglin duglin deleted the versionEventType branch November 30, 2018 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants