Skip to content

Conversation

@yordis
Copy link
Contributor

@yordis yordis commented Aug 28, 2025

Fixes #1364

yordis added a commit to yordis/spec that referenced this pull request Aug 28, 2025
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

fixes cloudevents#1366
@yordis yordis force-pushed the yordis/fixes-1364 branch from f9930bd to c90c905 Compare August 28, 2025 19:56
@yordis yordis marked this pull request as ready for review August 28, 2025 19:57
@duglin
Copy link
Collaborator

duglin commented Aug 28, 2025

/cc @deissnerk

yordis added a commit to yordis/spec that referenced this pull request Aug 28, 2025
Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

fixes cloudevents#1366
@yordis yordis force-pushed the yordis/fixes-1364 branch from c90c905 to cc57161 Compare August 28, 2025 20:01
@duglin
Copy link
Collaborator

duglin commented Aug 28, 2025

Minor thing, but can you make the events in your example full CEs? Meaning add the missing required attributes?

yordis added a commit to yordis/spec that referenced this pull request Aug 28, 2025
fixes cloudevents#1366

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis force-pushed the yordis/fixes-1364 branch from cc57161 to 20d919e Compare August 28, 2025 20:24
yordis added a commit to yordis/spec that referenced this pull request Aug 28, 2025
fixes cloudevents#1366

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis force-pushed the yordis/fixes-1364 branch from 20d919e to a1a3b79 Compare August 28, 2025 20:25
@yordis yordis requested a review from duglin August 28, 2025 20:26
yordis added a commit to yordis/spec that referenced this pull request Sep 2, 2025
fixes cloudevents#1366

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis requested a review from duglin September 2, 2025 16:00
@duglin
Copy link
Collaborator

duglin commented Sep 2, 2025

LGTM
/cc @deissnerk for review

@deissnerk
Copy link
Contributor

LGTM
Thanks @yordis !

fixes cloudevents#1366

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@duglin
Copy link
Collaborator

duglin commented Sep 4, 2025

Approved on the Sep 4 call

@duglin duglin merged commit 1f91d35 into cloudevents:main Sep 4, 2025
2 checks passed
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.

Scope of "sequence" extension

3 participants