Skip to content

Revert "Revert "refactor: use cesdk"" - #400

Merged
grant merged 5 commits into
masterfrom
revert-399-revert-391-grant_cloudevent_sdk
Dec 2, 2021
Merged

grant merged 5 commits into
masterfrom
revert-399-revert-391-grant_cloudevent_sdk

Conversation

@grant

@grant grant commented Dec 2, 2021 •

Copy link
Copy Markdown
Contributor

Reverts #399 (roll forward).

See PR description for details: #391 (comment)

WANT_LGTM=@matthewrobertson

@grant grant self-assigned this Dec 2, 2021
@google-cla google-cla Bot added the cla: yes label Dec 2, 2021
@matthewrobertson

Copy link
Copy Markdown
Member

We should pull in an updated version of the CESDK that includes the changes from cloudevents/sdk-javascript#446

Comment thread src/cloud_events.ts Resolved
Comment thread src/functions.ts Resolved
Comment thread src/functions.ts Resolved
Comment thread test/function_wrappers.ts Resolved
Comment thread test/integration/cloud_event.ts Resolved
Comment thread test/integration/cloud_event.ts Resolved
Comment thread src/function_wrappers.ts Resolved
Comment thread docs/generated/api.json Resolved
@grant

grant commented Dec 2, 2021

Copy link
Copy Markdown
Contributor Author

We should pull in an updated version of the CESDK that includes the changes from cloudevents/sdk-javascript#446

Alright by me. I've asked lance to see if we can get this in.

grant added 4 commits December 1, 2021 21:37
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
Signed-off-by: Grant Timmerman <timmerman+devrel@google.com>
@grant grant mentioned this pull request Dec 2, 2021
@grant

grant commented Dec 2, 2021

Copy link
Copy Markdown
Contributor Author

Thanks for the review @matthewrobertson. PTAL :)

@matthewrobertson

Copy link
Copy Markdown
Member

LGTM

@grant

grant commented Dec 2, 2021

Copy link
Copy Markdown
Contributor Author

Merging with Matt's LGTM. Annie already reviewed the other PR.

@grant
grant merged commit 8aef26e into master Dec 2, 2021
@grant
grant deleted the revert-399-revert-391-grant_cloudevent_sdk branch December 2, 2021 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants