Skip to content

Update examples to latest(2.x) sdk - #206

Merged
lance merged 1 commit into
cloudevents:masterfrom
lholmquist:187-update-examples
Jun 9, 2020
Merged

lance merged 1 commit into
cloudevents:masterfrom
lholmquist:187-update-examples

Conversation

@lholmquist

Copy link
Copy Markdown
Contributor

This PR starts to update the examples in examples to use the latest released cloudevents-sdk(which is 2.x atm)

fixes #187

@lholmquist
lholmquist force-pushed the 187-update-examples branch 2 times, most recently from 1105cb2 to b26b15b Compare June 4, 2020 21:09
@lholmquist
lholmquist marked this pull request as ready for review June 4, 2020 21:09
@lholmquist

Copy link
Copy Markdown
Contributor Author

looks like i can't add reviewers, so @lance @helio-frota @grant

@lance lance left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few suggestions, but otherwise LGTM. Thanks!

I think we should wait to land it though until 2.0.2.

Comment thread examples/typescript-ex/src/index.ts Resolved
Comment thread examples/typescript-ex/src/index.ts Resolved
Comment thread examples/typescript-ex/src/index.ts Resolved
Comment thread examples/typescript-ex/src/index.ts Resolved
@lance
lance marked this pull request as draft June 4, 2020 21:20
@lance

lance commented Jun 4, 2020

Copy link
Copy Markdown
Member

Converted to a draft until 2.0.2 lands

@lance

lance commented Jun 4, 2020

Copy link
Copy Markdown
Member

@lholmquist you will need to modify your commits with --signoff before we can merge this (a CNCF requirement).

@lholmquist
lholmquist force-pushed the 187-update-examples branch from 13bc973 to d764956 Compare June 9, 2020 17:57
Signed-off-by: Lucas Holmquist <lholmqui@redhat.com>
@lholmquist
lholmquist force-pushed the 187-update-examples branch from d764956 to 419c508 Compare June 9, 2020 17:59
@lholmquist
lholmquist marked this pull request as ready for review June 9, 2020 17:59
@lholmquist
lholmquist requested a review from lance June 9, 2020 17:59
@lholmquist

Copy link
Copy Markdown
Contributor Author

I've updated to the 2.0.2 version now

@lance lance left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM - ty!

@lance
lance merged commit dcb3c4e into cloudevents:master Jun 9, 2020
@helio-frota
helio-frota self-requested a review June 10, 2020 16:01
@lance lance mentioned this pull request Jul 7, 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.

Examples should be updated with new API

2 participants