Closed JmPotato closed 20 minutes ago
Attention: Patch coverage is 84.70588%
with 13 lines
in your changes missing coverage. Please review.
Project coverage is 77.37%. Comparing base (
e7c9d15
) to head (0a4f215
).:exclamation: Current head 0a4f215 differs from pull request most recent head bc2c7d3
Please upload reports for the commit bc2c7d3 to get more accurate results.
@okJiang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
@okJiang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
@okJiang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
nit: Can you add to the pr description the log display for different log settings?
Pasted. PTAL.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: HuSharp, JmPotato, niubell, okJiang, rleungx
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What problem does this PR solve?
Issue Number: ref #8305.
What is changed and how does it work?
Check List
Tests
Release note