Skip to content

docs: minor typo and grammar fixups#3984

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
nicktrav:nickt.doc-updates
Jul 30, 2018
Merged

docs: minor typo and grammar fixups#3984
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
nicktrav:nickt.doc-updates

Conversation

@nicktrav
Copy link
Copy Markdown
Contributor

Minor fixups for the xDS protocol documentation.

Signed-off-by: Nick Travers n.e.travers@gmail.com

Risk Level: Low
Docs Changes: Spelling and grammar in XDS_PROTOCOL.md

Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

As long as you are cleaning up grammar, this sentence still reads pretty confusingly to me. I think it would read better with what we had before, but in two sentences.

"For EDS/RDS, the management server does not need to supply every requested
resource and may also supply additional, unrequested resources. resource_names is only a hint." ? @htuch for thoughts also on this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

np, can update.

Minor fixups for the xDS protocol documentation.

Signed-off-by: Nick Travers <n.e.travers@gmail.com>
@nicktrav nicktrav force-pushed the nickt.doc-updates branch from 702bc06 to 5d8b9b9 Compare July 30, 2018 04:45
@mattklein123 mattklein123 self-assigned this Jul 30, 2018
@htuch htuch self-assigned this Jul 30, 2018
@mattklein123 mattklein123 merged commit ed131cf into envoyproxy:master Jul 30, 2018
@nicktrav nicktrav deleted the nickt.doc-updates branch September 6, 2020 21:52
@nicktrav nicktrav restored the nickt.doc-updates branch September 6, 2020 21:52
@nicktrav nicktrav deleted the nickt.doc-updates branch September 6, 2020 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants