Skip to content

Update k8s.io/docs/tutorials/hello-minikube#18673

Merged
k8s-ci-robot merged 2 commits intokubernetes:masterfrom
Kartik494:updateDoc_Addon
Jan 16, 2020
Merged

Update k8s.io/docs/tutorials/hello-minikube#18673
k8s-ci-robot merged 2 commits intokubernetes:masterfrom
Kartik494:updateDoc_Addon

Conversation

@Kartik494
Copy link
Copy Markdown
Contributor

@Kartik494 Kartik494 commented Jan 14, 2020

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 14, 2020
@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Jan 14, 2020
@php-coder
Copy link
Copy Markdown
Contributor

php-coder commented Jan 14, 2020

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 14, 2020
php-coder
php-coder reviewed Jan 14, 2020
View reviewed changes
content/en/docs/tutorials/hello-minikube.md Outdated

NAME TYPE CLUSTER-IP EXTERNAL-IP PORT(S) AGE
service/heapster ClusterIP 10.96.241.45 <none> 80/TCP 26s
service/metrics-server ClusterIP 10.96.241.45 <none> 80/TCP 26s
Copy link
Copy Markdown
Contributor

@php-coder php-coder Jan 14, 2020

Choose a reason for hiding this comment

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

Perhaps, we could improve alignment here? Or it isn't an issue in a rendered form?

Copy link
Copy Markdown
Contributor

@sftim sftim Jan 14, 2020

Choose a reason for hiding this comment

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

@php-code the deploy preview shows how this will look (see the deploy/netlify section of the checks list for a link to the preview).

Seems like your concern is valid.
/lgtm cancel

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 14, 2020
@php-coder php-coder mentioned this pull request Jan 14, 2020
Merged
@php-coder
Copy link
Copy Markdown
Contributor

php-coder commented Jan 15, 2020

@Kartik494 Only one really small improvement is needed. Could you address it when you have time? Thanks!

@Kartik494
Copy link
Copy Markdown
Contributor Author

Kartik494 commented Jan 16, 2020

@php-coder @sftim i have addressed the changes.Please check
Thanks

php-coder
php-coder approved these changes Jan 16, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@php-coder php-coder left a comment

Choose a reason for hiding this comment

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

Thanks!

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 16, 2020
@sftim
Copy link
Copy Markdown
Contributor

sftim commented Jan 16, 2020

/approve
Thanks @Kartik494 !

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Jan 16, 2020

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sftim

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details 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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 16, 2020
@k8s-ci-robot k8s-ci-robot merged commit 9b9910c into kubernetes:master Jan 16, 2020
wawa0210 pushed a commit to wawa0210/website that referenced this pull request Mar 2, 2020
@Kartik494 @wawa0210
Update k8s.io/docs/tutorials/hello-minikube (kubernetes#18673)
8e47d2a 
* Update k8s.io/docs/tutorials/hello-minikube

* Update hello-minikube.md
This was referenced Mar 2, 2020
Closed
Closed
@Kartik494 Kartik494 mentioned this pull request Mar 18, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sftim sftim Awaiting requested review from sftim

@zparnold zparnold Awaiting requested review from zparnold

1 more reviewer

@php-coder php-coder php-coder approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@php-coder php-coder

@sftim sftim

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Kartik494 @php-coder @sftim @k8s-ci-robot