Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

docs: Add arch link to README file#137

Closed
grahamwhaley wants to merge 1 commit intokata-containers:masterfrom
grahamwhaley:20190112_test_codeowners
Closed

docs: Add arch link to README file#137
grahamwhaley wants to merge 1 commit intokata-containers:masterfrom
grahamwhaley:20190112_test_codeowners

Conversation

@grahamwhaley
Copy link
Copy Markdown
Contributor

Add a link in the rather sparse top level proxy README
file off to the relevant architecture documentation.

Fixes: #136

Signed-off-by: Graham Whaley graham.whaley@intel.com

@grahamwhaley
Copy link
Copy Markdown
Contributor Author

Adding this PR in prep for #135 landing. Hence, DNM right now.

@opendev-zuul
Copy link
Copy Markdown

opendev-zuul bot commented Jan 12, 2019

Build failed (third-party-check pipeline) integration testing with
OpenStack. For information on how to proceed, see
http://docs.openstack.org/infra/manual/developers.html#automated-testing

@jodh-intel
Copy link
Copy Markdown

jodh-intel commented Jan 14, 2019

lgtm

Approved with PullApprove Approved with PullApprove

Add a link in the rather sparse top level proxy README
file off to the relevant architecture documentation.

Fixes: kata-containers#136

Signed-off-by: Graham Whaley <graham.whaley@intel.com>
@grahamwhaley
Copy link
Copy Markdown
Contributor Author

Now #135 has landed, I pushed a change, but don't see anything relating to the CODEOWNERS file - I suspect either:

  • the CODEOWNERS does not support github teams (and I could guess why maybe)
  • it only checks the CODEOWNERS when the PR is opened

So, let me close this and re-open a new one, and we'll see if it is the same...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants