Skip to content

1541 update coverage report config#1746

Merged
wyli merged 11 commits intomasterfrom
1541-coverage-config
Mar 11, 2021
Merged

1541 update coverage report config#1746
wyli merged 11 commits intomasterfrom
1541-coverage-config

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented Mar 11, 2021

Fixes #1541

Description

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).

wyli added 2 commits March 11, 2021 10:45
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli requested review from Nic-Ma, ericspod and rijobro March 11, 2021 11:02
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Mar 11, 2021

Hi @wyli ,

I think this PR is still a draft, is it ready for review?

Thanks.

@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Mar 11, 2021

Hi @wyli ,

I think this PR is still a draft, is it ready for review?

Thanks.

yes it works locally, I'm waiting for the CI results to confirm it and then it's ready to merge

@Nic-Ma Nic-Ma requested review from IsaacYangSLA and madil90 March 11, 2021 11:21
@wyli wyli force-pushed the 1541-coverage-config branch from c208004 to 78f3a3f Compare March 11, 2021 14:20
wyli added 4 commits March 11, 2021 15:42
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli marked this pull request as ready for review March 11, 2021 18:11
Signed-off-by: Wenqi Li <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Mar 11, 2021

@wyli wyli merged commit 401ea29 into master Mar 11, 2021
@wyli wyli deleted the 1541-coverage-config branch March 11, 2021 22:42
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.

inaccurate code coverage report for multiprocess executions

2 participants