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

Switch to testr package #416

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Switch to testr package #416

merged 1 commit into from
Aug 28, 2023

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Aug 28, 2023

The logr/testing package was deprecated in favor of the logr/testr package. There are no changes in functionality as the logr/testing package is implemented using the logr/testr package.

The logr/testing package was deprecated in favor of the logr/testr
package. There are no changes in functionality as the logr/testing
package is implemented using the logr/testr package.

Signed-off-by: Scott Andrews <andrewssc@vmware.com>
@codecov
Copy link

codecov bot commented Aug 28, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (2772a61) 60.62% compared to head (f28a77a) 60.62%.

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #416   +/-   ##
=======================================
  Coverage   60.62%   60.62%           
=======================================
  Files          24       24           
  Lines        2334     2334           
=======================================
  Hits         1415     1415           
  Misses        835      835           
  Partials       84       84           
Files Changed Coverage Δ
testing/reconciler.go 0.00% <0.00%> (ø)
testing/subreconciler.go 0.00% <0.00%> (ø)
testing/webhook.go 0.00% <0.00%> (ø)

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

@scothis scothis merged commit 55557b2 into vmware-labs:main Aug 28, 2023
1 check passed
@scothis scothis deleted the testr branch August 28, 2023 17:30
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