-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
coverage: Use upstream coverage collect script #28802
Conversation
2287ed9
to
361041f
Compare
|
7fa5c17
to
b081f1c
Compare
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
d62425d
to
041711e
Compare
a50959b
to
f55ef56
Compare
f55ef56
to
f7af08c
Compare
/retest mobile |
/retest mobile again |
/retest mobile to be sure (network tranients afaict) |
/retest mobile |
8936511
to
83a16fe
Compare
eb50128
to
970456f
Compare
Signed-off-by: Ryan Northey <ryan@synca.io>
970456f
to
c1f5686
Compare
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io> Signed-off-by: phlax <phlax@users.noreply.github.com>
)" This reverts commit 8004b60. Signed-off-by: Ryan Northey <ryan@synca.io>
…proxy#28802)"" This reverts commit 98adf91. Signed-off-by: Ryan Northey <ryan@synca.io>
)" This reverts commit 8004b60.
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io> Signed-off-by: phlax <phlax@users.noreply.github.com>
Our custom collect script should no longer be necessary as upstream now has the support for which it was added
This speeds coverage up by ~25% in my testing, and uses a lot less disk space
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]