-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Minor documentation cleanup #7826
Merged
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
1f47a5f
clarify link
jsoref 47f51af
Add section headers
jsoref 802e279
console blocks
jsoref 027d7ca
grpc example json was not valid
jsoref 3267048
multi-tls update text
jsoref 4bf63aa
add whitespace around parens
jsoref 10e1baa
grammar
jsoref 6ef4478
grammar
jsoref 2d6e7a7
Use bullets for choices
jsoref b9c9b97
ingress-controller
jsoref 766705d
drop stray paren
jsoref 8927c1b
OAuth is a brand and needs an article here
jsoref 3e0cc91
Indent text under numbered lists
jsoref cd578eb
use e.g.
jsoref 00dc586
Document that customer header config maps changes do not trigger updates
jsoref cdac21c
article
jsoref bbaec16
period
jsoref c46a2bf
infinitive verb + period
jsoref abbd832
clarify that the gRPC server is responsible for listening for TCP tra…
jsoref 82af094
avoid using ; and reword
jsoref f5512de
whitespace
jsoref 0158859
brand: gRPC
jsoref 77c4185
only-does is the right form
jsoref 56b360a
spelling: GitHub
jsoref 7e66194
punctuation
jsoref e2c3d94
drop stray `to`
jsoref b995864
sentence
jsoref cf6da8e
backticks
jsoref 1c528d3
fix link
jsoref 50444a9
Improve readability of compare/vs
jsoref ec65187
Renumber list
jsoref 4e49830
punctuation
jsoref d5a748f
Favor Ingress-NGINX and Ingress NGINX
jsoref e53694a
Simplify custom header restart text
jsoref edd9a3b
Undo typo damage
jsoref File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Can we use the phrase "ingress-nginx 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.
Certainly. Note that a whole bunch of phrases are used, if that's your preferred one, I'll try to apply it consistently across the markdown files.
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.
Please do. It was talked about and I am certain we want consistent "ingress-NGINX controller" everywhere. "NGINX" in caps or not is left to you. Thanks for the improvement. Helps.