-
Notifications
You must be signed in to change notification settings - Fork 22
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
refactor: Make available complete image info, instead of UUID only #489
base: main
Are you sure you want to change the base?
Conversation
We plan to make decisions based on some of the information.
84f076c
to
236044f
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #489 +/- ##
==========================================
+ Coverage 33.46% 34.70% +1.23%
==========================================
Files 17 18 +1
Lines 2235 2239 +4
==========================================
+ Hits 748 777 +29
+ Misses 1467 1442 -25
Partials 20 20 ☔ View full report in Codecov by Sentry. |
…only Refactor further; add unit tests with mocks
I made substantial changes to |
…only Use longer name for receivers
What this PR does / why we need it:
We plan to make decisions based on the complete image info.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
How Has This Been Tested?:
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration and test output
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note: