Closed okJiang closed 2 weeks ago
Attention: Patch coverage is 77.92208%
with 17 lines
in your changes missing coverage. Please review.
Project coverage is 77.32%. Comparing base (
9dff6e6
) to head (44c9cb0
).:exclamation: Current head 44c9cb0 differs from pull request most recent head a8685db
Please upload reports for the commit a8685db to get more accurate results.
Can you take a look again~ @rleungx @HuSharp @JmPotato
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: JmPotato, rleungx
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/merge
@JmPotato: We have migrated to builtin LGTM
and approve
plugins for reviewing.
👉 Please use /approve
when you want approve this pull request.
The changes announcement: Proposal: Strengthen configuration change approval.
What problem does this PR solve?
Issue Number: Ref #4399
What is changed and how does it work?
Check List
Tests
Release note