Skip to content
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

Typo #5998

Merged
merged 1 commit into from
Oct 22, 2017
Merged

Typo #5998

merged 1 commit into from
Oct 22, 2017

Conversation

karthequian
Copy link
Contributor

@karthequian karthequian commented Oct 22, 2017

I believe this should be "users" not "uses"

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.9 Features: set Milestone to 1.9 and Base Branch to release-1.9
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.

Please delete this note before submitting the pull request.

NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box. This is a temporary workaround to address a known issue with GitHub.

Allow edits from maintainers checkbox


This change is Reviewable

I believe this should be "users" not "uses"
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 22, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

Deploy preview ready!

Built with commit f639e06

https://deploy-preview-5998--kubernetes-io-master-staging.netlify.com

Copy link
Contributor

@tengqm tengqm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 22, 2017
@heidecke
Copy link
Contributor

I don't think that this is actually a typo. "...separation of uses" is more accurate than a "separation of users" in this context.

Namespace is probably a good next term to add to the glossary though so that we have a standard concept definition. @Bradamant3 may already be working that.

@xiangpengzhao
Copy link
Contributor

Namespace is probably a good next term to add to the glossary though so that we have a standard concept definition.

FYI: #5993

cc @chenopis

@heckj
Copy link
Contributor

heckj commented Oct 22, 2017

/lgtm

@heckj heckj merged commit 62e5e45 into kubernetes:master Oct 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants