-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
@luc99hen: GitHub didn't allow me to assign the following users: your_reviewer. Note that only openyurtio members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @rambohe-ch |
Codecov Report
@@ Coverage Diff @@
## master #97 +/- ##
==========================================
+ Coverage 21.21% 25.27% +4.05%
==========================================
Files 21 25 +4
Lines 2550 2742 +192
==========================================
+ Hits 541 693 +152
- Misses 1932 1951 +19
- Partials 77 98 +21
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: luc99hen, rambohe-ch The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@luc99hen it's better rebase master |
What type of PR is this?
What this PR does / why we need it:
add np webhook unit test & complement np controller test
Which issue(s) this PR fixes:
Fixes #
#73
Special notes for your reviewer:
Does this PR introduce a user-facing change?
other Note