Skip to content

Conversation

@afrittoli
Copy link
Contributor

@afrittoli afrittoli commented Aug 23, 2022

Add initial jsonschemas for all of the events.

These json schemas are automatically generated through the
go types defined in the new cdevents/go-sdk, using the
library github.com/invopop/jsonschema, see
cdevents/sdk-go#6 for details.

Signed-off-by: Andrea Frittoli [email protected]

@afrittoli afrittoli force-pushed the schemas branch 2 times, most recently from 4b40f83 to a3d2639 Compare August 25, 2022 13:31
@afrittoli afrittoli changed the title Add jsonschema for a few of the events Add jsonschema for the events Aug 25, 2022
Copy link
Contributor

@e-backmark-ericsson e-backmark-ericsson left a comment

Choose a reason for hiding this comment

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

Provided some comments for artifactpackaged.json. Some of those are valid for all event types more or less.

Add initial jsonschemas for all of the events.

These json schemas are automatically generated through the
go types defined in the new cdevents/go-sdk, using the
library github.com/invopop/jsonschema, see
cdevents/sdk-go#6 for details.

Signed-off-by: Andrea Frittoli <[email protected]>
Signed-off-by: Andrea Frittoli <[email protected]>
Copy link
Contributor

@e-backmark-ericsson e-backmark-ericsson left a comment

Choose a reason for hiding this comment

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

Haven't reviewed this in too much detail for each event type, but overall it looks good so it would be ok to merge as a first version of the schemas.

@afrittoli afrittoli merged commit 7906c90 into cdevents:main Sep 13, 2022
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