Skip to content

Remove "TBD" comment about extensions#162

Merged
duglin merged 1 commit into
cloudevents:masterfrom
duglin:removeTBD
Apr 19, 2018
Merged

Remove "TBD" comment about extensions#162
duglin merged 1 commit into
cloudevents:masterfrom
duglin:removeTBD

Conversation

@duglin
Copy link
Copy Markdown
Collaborator

@duglin duglin commented Apr 15, 2018

We don't need the TBD since each mapping/serialization doc we produce
should state how extensions should appear. E.g. #148

Closes: #125

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

We don't need the TBD since each mapping/serialization doc we produce
should state how extensions should appear. E.g. cloudevents#148

Closes: cloudevents#125

Signed-off-by: Doug Davis <dug@us.ibm.com>
@clemensv
Copy link
Copy Markdown
Contributor

LGTM

@duglin
Copy link
Copy Markdown
Collaborator Author

duglin commented Apr 17, 2018

I consider this more of a clean-up/syntax thing for the spec. If we can get some more reviews on this before the meeting then we can resolve it quickly. thanks

@duglin
Copy link
Copy Markdown
Collaborator Author

duglin commented Apr 19, 2018

Approved on 4/19 call

@duglin duglin merged commit 9ca339d into cloudevents:master Apr 19, 2018
@duglin duglin deleted the removeTBD branch April 19, 2018 19:32
clemensv pushed a commit to clemensv/spec that referenced this pull request May 16, 2018
We don't need the TBD since each mapping/serialization doc we produce
should state how extensions should appear. E.g. cloudevents#148

Closes: cloudevents#125

Signed-off-by: Doug Davis <dug@us.ibm.com>
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