-
Notifications
You must be signed in to change notification settings - Fork 115
Conversation
See the changes at https://github.com/adrianludwin/hierarchical-namespaces/tree/releases. /assign @rjbez17 |
docs/contributing.md
Outdated
from the command line, [as documented | ||
here](https://kubernetes.io/docs/reference/access-authn-authz/rbac/#referring-to-subjects). | ||
|
||
I *think* you can also use client certificate requests but I haven't tried that |
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.
nit: I -> We
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
a few nits but lgtm if you decide to skip them (remove hold). /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adrianludwin, rjbez17 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 |
A variety of cleanups: * Move CONTRIBUTING.md to the /docs directory and actually add the useful information that was previously in the README. * Update the readme and user guide to explain the difference between the new and old repo. * Update the Krew documentation so that it doesn't need to be updated every release, and to point to the new repo.
/hold cancel |
/lgtm |
A variety of cleanups:
useful information that was previously in the README.
new and old repo.
every release, and to point to the new repo.