Closed rleungx closed 2 weeks ago
Attention: Patch coverage is 14.28571%
with 12 lines
in your changes missing coverage. Please review.
Project coverage is 77.31%. Comparing base (
463aee9
) to head (e541372
).
@okJiang: adding LGTM is restricted to approvers and reviewers in OWNERS files.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: HuSharp, okJiang
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Timeline:
2024-06-14 09:56:40.146028361 +0000 UTC m=+717754.199340283
: :ballot_box_with_check: agreed by HuSharp./merge
@rleungx: 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 #1919.
What is changed and how does it work?
Check List
Tests
Release note