Closed nacx closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: No coverage uploaded for pull request base (
main@11a2002
). Click here to learn what that means.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
/hold
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nacx, Shikugawa
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm.
/unhold
It's been 1 week and there were no objections, so let's have this merged. We can always add back any maintainer that wants to be a code owner.
Update the code owners to the current active people, so that code reviews can happen in a timely manner.
It mgirates from the OWNERS file to GitHub CODEOWNERS. Once this is merged, a follow-up change will be done in
istio/test-infra
to update the Ci requirements accordingly.@BrenoDeMedeiros @liminw @lizan @sawlanipradeep @incfly @Shikugawa you were listed in the OWNERS file, but as far as I know you're not active anymore in the project. Please leave a comment if you want to be kept in the CODEOWNERS file and I would be happy to add you before merging the PR. Thank you! :)