Skip to content

test: improve docs and robustness of coverage script.#14021

Merged
lizan merged 2 commits intoenvoyproxy:masterfrom
htuch:foo
Nov 19, 2020
Merged

test: improve docs and robustness of coverage script.#14021
lizan merged 2 commits intoenvoyproxy:masterfrom
htuch:foo

Conversation

@htuch
Copy link
Copy Markdown
Member

@htuch htuch commented Nov 13, 2020

I ended up burning several hours dealing with issues around this the
other day, I'm hoping others don't :)

Signed-off-by: Harvey Tuch [email protected]

I ended up burning several hours dealing with issues around this the
other day, I'm hoping others don't :)

Signed-off-by: Harvey Tuch <[email protected]>
@htuch htuch requested a review from lizan November 13, 2020 20:30
@htuch htuch requested a review from asraa November 13, 2020 20:30
Copy link
Copy Markdown
Contributor

@asraa asraa left a comment

Choose a reason for hiding this comment

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

Thank you! Great to see the script check on top of the docs.

Signed-off-by: Harvey Tuch <[email protected]>
@lizan lizan merged commit 6e08670 into envoyproxy:master Nov 19, 2020
andreyprezotto pushed a commit to andreyprezotto/envoy that referenced this pull request Nov 24, 2020
I ended up burning several hours dealing with issues around this the
other day, I'm hoping others don't :)

Signed-off-by: Harvey Tuch <[email protected]>
qqustc pushed a commit to qqustc/envoy that referenced this pull request Nov 24, 2020
I ended up burning several hours dealing with issues around this the
other day, I'm hoping others don't :)

Signed-off-by: Harvey Tuch <[email protected]>
Signed-off-by: Qin Qin <[email protected]>
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