Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Apply v0.1.1 to main #88

Merged
merged 3 commits into from
Oct 26, 2022
Merged

Apply v0.1.1 to main #88

merged 3 commits into from
Oct 26, 2022

Conversation

afrittoli
Copy link
Contributor

Changes

Apply the v0.1.1 changes to main too.
Since there has been no other development on main yet, we can apply these commits as they are.
Once this is merged I will switch main to v0.2.0-draft.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

The repository is an optional field in branch and change events,
both in the specification as well as in the SDK.

This is because the source includes the repository information already.
While there is value in specifying the repository reference separated
from the SCM system URL, it should not be mandatory to do so.

The schema files for branch and change events specify the repository
as a mandatory field. This was an oversight, which is fixed here.

Signed-off-by: Andrea Frittoli <[email protected]>
Signed-off-by: Andrea Frittoli <[email protected]>
The schema for branch and change events was updated to make the
repository non-mandatory.

Update the event version in the spec.
Update the valid spec version in the schema, so that events 0.1.1
are only available in spec 0.1.1.

Signed-off-by: Andrea Frittoli <[email protected]>
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