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

doc: fix formatting of regexes, disconnected comment #8285

Merged

Conversation

zuercher
Copy link
Member

Some example regexes were rendered incorrectly on envoyproxy.io. I chose
to try to find all the example regexes and make them literal text. This
way the regexes are syntactically correct both in the raw proto and on
the doc site. Also found an H2 protocol option with a missing paragraph
due to a blank line vs. empty comment line.

Risk: low, comment-only
Testing: n/a
Docs: updated
Release Notes: n/a

Signed-off-by: Stephan Zuercher [email protected]

Some example regexes were rendered incorrectly on envoyproxy.io. I chose
to try to find all the example regexes and make them literal text. This
way the regexes are syntactically correct both in the raw proto and on
the doc site. Also found an H2 protocol option with a missing paragraph
due to a blank line vs. empty comment line.

Risk: low, comment-only
Testing: n/a
Docs: updated
Release Notes: n/a

Signed-off-by: Stephan Zuercher <[email protected]>
@zuercher
Copy link
Member Author

/retest

@repokitteh-read-only
Copy link

🔨 rebuilding ci/circleci: release (failed build)

🐱

Caused by: a #8285 (comment) was created by @zuercher.

see: more, trace.

@zuercher
Copy link
Member Author

ping @envoyproxy/api-shepherds (formatting-only comment changes)

Copy link
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!

@mattklein123 mattklein123 merged commit 35b459f into envoyproxy:master Sep 19, 2019
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Sep 24, 2019
Some example regexes were rendered incorrectly on envoyproxy.io. I chose
to try to find all the example regexes and make them literal text. This
way the regexes are syntactically correct both in the raw proto and on
the doc site. Also found an H2 protocol option with a missing paragraph
due to a blank line vs. empty comment line.

Risk: low, comment-only
Testing: n/a
Docs: updated
Release Notes: n/a

Signed-off-by: Stephan Zuercher <[email protected]>
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Oct 4, 2019
Some example regexes were rendered incorrectly on envoyproxy.io. I chose
to try to find all the example regexes and make them literal text. This
way the regexes are syntactically correct both in the raw proto and on
the doc site. Also found an H2 protocol option with a missing paragraph
due to a blank line vs. empty comment line.

Risk: low, comment-only
Testing: n/a
Docs: updated
Release Notes: n/a

Signed-off-by: Stephan Zuercher <[email protected]>
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Oct 4, 2019
Some example regexes were rendered incorrectly on envoyproxy.io. I chose
to try to find all the example regexes and make them literal text. This
way the regexes are syntactically correct both in the raw proto and on
the doc site. Also found an H2 protocol option with a missing paragraph
due to a blank line vs. empty comment line.

Risk: low, comment-only
Testing: n/a
Docs: updated
Release Notes: n/a

Signed-off-by: Stephan Zuercher <[email protected]>
@zuercher zuercher deleted the zuercher_fix_doc_formatting branch November 7, 2019 18:48
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