Skip to content

Polish#592

Merged
scothis merged 2 commits intoreconcilerio:mainfrom
mamachanko:topic/mamachanko/main/polish
Feb 10, 2025
Merged

Polish#592
scothis merged 2 commits intoreconcilerio:mainfrom
mamachanko:topic/mamachanko/main/polish

Conversation

@mamachanko
Copy link
Contributor

  • refactor: use ptr instead of the deprecated pointer
  • refactor: use _ for unused argument

Signed-off-by: Max Brauer <mbrauer@vmware.com>
Signed-off-by: Max Brauer <mbrauer@vmware.com>
@codecov
Copy link

codecov bot commented Feb 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.26%. Comparing base (df0175b) to head (a4b5477).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #592   +/-   ##
=======================================
  Coverage   58.26%   58.26%           
=======================================
  Files          33       33           
  Lines        3800     3800           
=======================================
  Hits         2214     2214           
  Misses       1492     1492           
  Partials       94       94           

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

@scothis scothis merged commit eb7b755 into reconcilerio:main Feb 10, 2025
4 checks passed
@mamachanko mamachanko deleted the topic/mamachanko/main/polish branch February 10, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants