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

Fix a typo and update one url #3041

Merged
merged 1 commit into from
Mar 28, 2017
Merged

Fix a typo and update one url #3041

merged 1 commit into from
Mar 28, 2017

Conversation

zhangxiaoyu-zidif
Copy link
Contributor

@zhangxiaoyu-zidif zhangxiaoyu-zidif commented Mar 28, 2017

Fix a typo and update one url

NOTE: Please check the “Allow edits from maintainers” box below to allow
reviewers fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.


This change is Reviewable

Fix a typo and update one url
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 28, 2017
@@ -20,7 +20,7 @@ Multiple kubeconfig files are allowed, if specified explicitly. At runtime they

## Related discussion

http://issue.k8s.io/1755
Copy link
Member

Choose a reason for hiding this comment

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

This is correct as is.

@chenopis chenopis self-assigned this Mar 28, 2017
@chenopis chenopis merged commit ba0ba49 into kubernetes:master Mar 28, 2017
@mikedanese
Copy link
Member

@chenopis why did you merge this when I had an outstanding comment?

@chenopis
Copy link
Contributor

chenopis commented Mar 28, 2017

@mikedanese Sorry. issue.k8s.io/1755 redirects to github.com/kubernetes/kubernetes/issues/1755 so I thought using the final URL was fine. Is it better to use the issue.k8s.io format? I can revert just that part if needed.

@chenopis
Copy link
Contributor

Since the compact form, http://issue.k8s.io/1755, makes things simpler, I'm reverting to that in PR #3047 and turning it into a clickable link.

@zhangxiaoyu-zidif zhangxiaoyu-zidif deleted the patch-19 branch March 29, 2017 03:24
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants