Closed yanggangtony closed 10 months ago
Merging #3581 (3a28669) into master (218a0b2) will not change coverage. The diff coverage is
n/a
.
:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@ Coverage Diff @@
## master #3581 +/- ##
=======================================
Coverage 55.86% 55.86%
=======================================
Files 216 216
Lines 20121 20121
=======================================
Hits 11240 11240
Misses 8276 8276
Partials 605 605
Flag | Coverage Ξ | |
---|---|---|
unittests | 55.86% <ΓΈ> (ΓΈ) |
Flags with carried forward coverage won't be shown. Click here to find out more.
/lgtm
/cc @Poor12 @lonelyCZ
ππ According @Poor12 's opinion. And these similar question have discussed in https://github.com/karmada-io/karmada/pull/3398.
Using o is more clearer.
So may i changes the options *DeInitOptions
to opt *DeInitOptions
?
ππ According @Poor12 's opinion. And these similar question have discussed in #3398.
Using o is more clearer.
So may i changes the
options *DeInitOptions
toopt *DeInitOptions
?
To be consistent with other components, we can use o
as the arg name.
options *DeInitOptions
-> o *DeInitOptions
, I think it's ok.
rebased.
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Poor12
The full list of commands accepted by this bot can be found here.
The pull request process is described here
What type of PR is this? /kind cleanup
What this PR does / why we need it: Do some operator cleanup.
Which issue(s) this PR fixes: Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: