-
Notifications
You must be signed in to change notification settings - Fork 65
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename CopyMethod None to CopyMethod Direct #99
Rename CopyMethod None to CopyMethod Direct #99
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks.
Expect(inst.Status.NextSyncTime).NotTo(BeNil()) | ||
}) | ||
// Test both None and Direct (results should be the same) | ||
for i := range directCopyMethodTypes { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤩
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnStrunk, tesshuflower 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 |
@tesshuflower Now that #98 is merged, I think this will pass if you rebase it. |
Signed-off-by: Tesshu Flower <tflower@redhat.com>
ebb9304
to
f4d62c5
Compare
Codecov Report
@@ Coverage Diff @@
## main #99 +/- ##
=======================================
- Coverage 75.4% 75.2% -0.3%
=======================================
Files 16 16
Lines 2044 2051 +7
=======================================
+ Hits 1542 1543 +1
- Misses 398 401 +3
- Partials 104 107 +3
|
/lgtm |
Signed-off-by: Tesshu Flower tflower@redhat.com
Describe what this PR does
Renames CopyMethod: None to CopyMethod: Direct. None will still work, but Direct has been added and will be the recommended name.
Is there anything that requires special attention?
Related issues:
#8