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

Restore support in ChildReconciler for unstructured resources #362

Merged
merged 1 commit into from
Apr 24, 2023
Merged

Restore support in ChildReconciler for unstructured resources #362

merged 1 commit into from
Apr 24, 2023

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Apr 24, 2023

Adding a duplicate test suite for ChildReconciler using unstructured parent and child resources.

Adding a duplicate test suite for ChildReconciler using unstructured
parent and child resources.

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

codecov bot commented Apr 24, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (492f3d0) 58.12% compared to head (5f36f0f) 58.12%.

❗ Current head 5f36f0f differs from pull request most recent head a442a3e. Consider uploading reports for the commit a442a3e to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #362   +/-   ##
=======================================
  Coverage   58.12%   58.12%           
=======================================
  Files          14       14           
  Lines        1956     1956           
=======================================
  Hits         1137     1137           
  Misses        741      741           
  Partials       78       78           
Impacted Files Coverage Δ
reconcilers/reconcilers.go 75.75% <100.00%> (ø)
testing/config.go 87.44% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@scothis scothis merged commit d3c6f93 into vmware-labs:main Apr 24, 2023
@scothis scothis deleted the unstructured-children branch April 24, 2023 18: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