Closed MundaneImmortal closed 3 months ago
Attention: Patch coverage is 0%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 54.71%. Comparing base (
56b874c
) to head (8765ca6
). Report is 4 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
...manager/controller/nodepool/nodepool_controller.go | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@MundaneImmortal Thanks for posting pull request. you need to fix the golangci-lint
errors. you can execute make lint
command locally to check golangci-lint
errors.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
@rambohe-ch Thanks for your feedback, I have fixed that and committed again.
/lgtm
@MundaneImmortal I am very appreciate for your first pull request, and i will merge it because all github actions(except codecov/patch) are passed.
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #1908
Special notes for your reviewer:
Does this PR introduce a user-facing change?
other Note