Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Streamline finalizer patching #434

Merged
merged 1 commit into from
Sep 12, 2023
Merged

Streamline finalizer patching #434

merged 1 commit into from
Sep 12, 2023

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Sep 12, 2023

Since the client can now handle duck type resources, we no longer need to cast the current resources back to the parent resource type.

Since the client can now handle duck type resources, we no longer need
to cast the current resources back to the parent resource type.

Signed-off-by: Scott Andrews <[email protected]>
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.29% ⚠️

Comparison is base (d7f3909) 61.01% compared to head (54c2a3e) 60.72%.

❗ Current head 54c2a3e differs from pull request most recent head e86fc5a. Consider uploading reports for the commit e86fc5a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #434      +/-   ##
==========================================
- Coverage   61.01%   60.72%   -0.29%     
==========================================
  Files          26       26              
  Lines        2529     2518      -11     
==========================================
- Hits         1543     1529      -14     
- Misses        899      902       +3     
  Partials       87       87              
Files Changed Coverage Δ
reconcilers/finalizer.go 84.48% <100.00%> (+3.32%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@scothis scothis merged commit 9b5e441 into vmware-labs:main Sep 12, 2023
1 check passed
@scothis scothis deleted the streamline-finalizers branch September 12, 2023 19:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants