-
Notifications
You must be signed in to change notification settings - Fork 3
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
Create a sample to show how to tag VMs as they are created. #18
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the only thing you need to do is change v1 to v2
cmd/sample/tag-created-vms/README.md
Outdated
"context" | ||
"log" | ||
|
||
cloudevents "github.com/cloudevents/sdk-go/v1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v1 -> v2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still v1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I knew I'd regret inlining that file, that didn't take long.
In the next release of cloudevents preview, this changes, we are going to break the contract on client, it will return a Result rather than an error, and you can check for ACK: cloudevents/sdk-go#425 |
dd1520a
to
0e213a7
Compare
I'd rather have the samples in samples/* directory. |
0e213a7
to
42688bc
Compare
/lgtm |
No description provided.