-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Revise main documentation page #8019
Conversation
/kind documentation |
/ok-to-test |
need to squash commits using bot before approve |
4f89ea1
to
483a122
Compare
BTW, Prow can preserve LGTM through squashes - see kubernetes/kubernetes#103096 for hints. |
/lgtm
Thanks,
; Long
…On Wed, 8 Dec, 2021, 7:38 AM Tim Bannister, ***@***.***> wrote:
BTW, Prow can preserve LGTM through squashes - see
kubernetes/kubernetes#103096
<kubernetes/kubernetes#103096> for hints.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#8019 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABGZVWXERZWB2SNTBZ3IYODUP24Z3ANCNFSM5JR7N6VA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
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.
Thanks!
24f497b
to
427bfa7
Compare
Thanks for the extra suggestions @tao12345666333 |
docs/index.md
Outdated
@@ -1,31 +1,35 @@ | |||
# Welcome | |||
# NGINX Ingress Controller |
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.
Alternatives:
- Leave
# Welcome
- Switch to
# Overview
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.
This will revise the heading on https://kubernetes.github.io/ingress-nginx/
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.
I like # Overview
- Revise to be more in line with the style guide for Kubernetes official docs - Avoid recommending that readers use `k8s.io` namespaced controller names for their own custom controller configuration. Co-authored-by: James Strong <strong.james.e@gmail.com>
100379c
to
f3476a9
Compare
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.
/lgtm
/triage accepted |
/approve Thanks :) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: longwuyuan, rikatz, sftim, tao12345666333 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 |
- Revise to be more in line with the style guide for Kubernetes official docs - Avoid recommending that readers use `k8s.io` namespaced controller names for their own custom controller configuration. Co-authored-by: James Strong <strong.james.e@gmail.com> Co-authored-by: James Strong <strong.james.e@gmail.com>
k8s.io
namespaced controller names for their own custom controller configuration.(Rather than a name like
k8s.io/ingress-nginx1
, readers should either use a private name:ingress-nginx1
, or use their own domain as a prefix).Rendered