Skip to content

Conversation

@nachocano
Copy link
Contributor

@nachocano nachocano commented May 8, 2019

Fixes #1285

Proposed Changes

  • Adding initial documentation for Event Registry feature

/label cherrypick-0.6

@knative-prow-robot knative-prow-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 8, 2019
@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label May 8, 2019
@knative-prow-robot knative-prow-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 8, 2019
@nachocano nachocano changed the title [WIP] Registry documentation Registry documentation May 9, 2019
@knative-prow-robot knative-prow-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 9, 2019
@samodell
Copy link
Contributor

Thanks @nachocano ! I'm taking a look at this now.

@nachocano
Copy link
Contributor Author

Thanks for all the comments @samodell . They were very useful!
Is ready for a second pass

@samodell
Copy link
Contributor

/lgtm
/approve

Thanks Nacho, this is great!

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label May 10, 2019
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nachocano, 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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot merged commit 9f86f17 into knative:master May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding documentation for Event Registry

4 participants