Skip to content

Merge - #2

Merged
grant merged 8 commits into
grant:masterfrom
cloudevents:master
Jul 29, 2020
Merged

grant merged 8 commits into
grant:masterfrom
cloudevents:master

Conversation

@grant

@grant grant commented Jul 29, 2020

Copy link
Copy Markdown
Owner

No description provided.

lholmquist and others added 8 commits July 27, 2020 13:19
BREAKING CHANGE:

This validates the value of the cloud event extension based on the spec, 
https://github.com/cloudevents/spec/blob/master/spec.md#type-system

Signed-off-by: Lucas Holmquist <lholmqui@redhat.com>
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
This commit removes two functions that are not used.

Signed-off-by: Lance Ball <lball@redhat.com>
fixes #278

Signed-off-by: Lucas Holmquist <lholmqui@redhat.com>
…281)

The schema incorrectly limits data values to only object and string. This is
incorrect, since JSON can be an array, boolean, a single number or null as well.

This commit modifies the schema to allow for array, boolean and null, and adds
tests.

Fixes: #280

Signed-off-by: Lance Ball <lball@redhat.com>
Signed-off-by: Lucas Holmquist <lholmqui@redhat.com>
@grant
grant merged commit 6097622 into grant:master Jul 29, 2020
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