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
Simplify the Github sample. #587
Conversation
3cdfa02
to
1e0043e
Compare
1e0043e
to
06c6c19
Compare
Should be RFAL, but ... /hold For possible discussion on Eventing slack. |
```shell | ||
ko apply --filename eventing/samples/github-source/subscription.yaml | ||
``` | ||
|
||
### Create Events |
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.
Can you verify the webhook before you create events?
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.
Done
deployed the [Subscription](#subscription), then continue using this | ||
section. If not, then you will need to look downstream yourself. | ||
eventing system by looking at our message dumper function logs. | ||
|
||
|
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.
Can you add the clean up section below?
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.
Done
This simplifies the Github sample as outlined in knative#586 to drop the concepts that aren't needed for the sample to work end-to-end. Fixes: knative#586 Fixes: knative#584 Fixes: knative#552 Fixes: knative#551 Progress on: knative#541
06c6c19
to
2bb1a3e
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mattmoor, samodell The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Co-Authored-By: mattmoor <mattmoor@google.com>
/hold cancel |
/lgtm |
This simplifies the Github sample as outlined in #586 to drop the concepts that aren't needed for the sample to work end-to-end.
Fixes: #586
Fixes: #584
Fixes: #552
Fixes: #551
Progress on: #541