Skip to content

Remove explicit extension attribute#228

Closed
evankanderson wants to merge 1 commit into
cloudevents:masterfrom
evankanderson:master
Closed

Remove explicit extension attribute#228
evankanderson wants to merge 1 commit into
cloudevents:masterfrom
evankanderson:master

Conversation

@evankanderson
Copy link
Copy Markdown
Contributor

@evankanderson evankanderson commented Jun 4, 2018

Remove explicit extensions attribute in favor of additional "bare" keys in the top-level context.

Ref: 2018-05-30 WG meeting:

https://docs.google.com/document/d/1OVF68rpuPK5shIHILK9JOqlZBbfe91RNzQ7u_P7YCDE/edit#

Signed-off-by: Evan Anderson argent@google.com

@evankanderson evankanderson force-pushed the master branch 2 times, most recently from 5eeff5e to 0f7b748 Compare June 4, 2018 21:29
Remove explicit `extensions` attribute in favor of additional "bare" keys in the top-level context.

Ref: 2018-05-30 WG meeting:

https://docs.google.com/document/d/1OVF68rpuPK5shIHILK9JOqlZBbfe91RNzQ7u_P7YCDE/edit#

Signed-off-by: Evan Anderson <argent@google.com>
@duglin
Copy link
Copy Markdown
Collaborator

duglin commented Jun 4, 2018

@evankanderson is this a dup of: #225 ?

@evankanderson
Copy link
Copy Markdown
Contributor Author

Maybe -- I realized I'd forgotten to squash the PR, and meant to cancel before hitting create.

#225 doesn't load for me right now.

@evankanderson
Copy link
Copy Markdown
Contributor Author

Yep, it's a dup. I didn't push on Thursday, and you got around to it while I was travelling on Friday.

Closing.

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.

2 participants