Skip to content

docs: add seealso section to gzip sandbox#16796

Merged
phlax merged 5 commits intoenvoyproxy:mainfrom
daixiang0:also
Jun 8, 2021
Merged

docs: add seealso section to gzip sandbox#16796
phlax merged 5 commits intoenvoyproxy:mainfrom
daixiang0:also

Conversation

@daixiang0
Copy link
Copy Markdown
Member

Signed-off-by: Long Dai long0dai@foxmail.com

Commit Message:

Add seealso section to gzip sandbox

Additional Description:
Risk Level: LOW
Testing:
Docs Changes: add seealso section.
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #16748
[Optional Deprecated:]
[Optional API Considerations:]

Signed-off-by: Long Dai <long0dai@foxmail.com>
@phlax
Copy link
Copy Markdown
Member

phlax commented Jun 3, 2021

looks good @daixiang0 - could we also add a link to the gzip api docs please

Signed-off-by: Long Dai <long0dai@foxmail.com>
Signed-off-by: Long Dai <long0dai@foxmail.com>
Signed-off-by: Long Dai <long0dai@foxmail.com>
@phlax
Copy link
Copy Markdown
Member

phlax commented Jun 7, 2021

rendered docs are here https://storage.googleapis.com/envoy-pr/16796/docs/index.html

@daixiang0 could you fix format please

@phlax
Copy link
Copy Markdown
Member

phlax commented Jun 7, 2021

probably you want to adjust your editor so these dont happen

ERRORS found
1 files with missing newline
0 files with mixed tabs and spaces
0 files with trailing whitespace

Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

almost there @daixiang0 a few nits below...

Signed-off-by: Long Dai <long0dai@foxmail.com>
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @daixiang0

@phlax phlax merged commit f25fbb2 into envoyproxy:main Jun 8, 2021
@daixiang0 daixiang0 deleted the also branch June 8, 2021 05:49
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Long Dai <long0dai@foxmail.com>
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.

Add "see also" section to gzip sandbox

3 participants