-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix(pkg/repository/maintenance): handle when there's no container status #8271
Conversation
Hi @mateusoliveira43 , sure, I've updated the comments and the error |
if you're like me, you'd notice rust would have made the problem much more obvious in the first place 😅 |
@mcluseau yeah, Go errors are almost unreadable, thanks for taking time to fix this! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8271 +/- ##
==========================================
+ Coverage 58.99% 59.00% +0.01%
==========================================
Files 367 367
Lines 38847 38857 +10
==========================================
+ Hits 22918 22928 +10
Misses 14467 14467
Partials 1462 1462 ☔ View full report in Codecov by Sentry. |
@mcluseau please add changelog |
Add changelog here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO. but not blocked on it.
…r statuses Signed-off-by: Mikaël Cluseau <[email protected]>
@mcluseau |
2 weeks looked like a lazy consensus to me, but ok, I won't touch those buttons again 😅 |
We are creating release of 1.15, let's merge this PR after 1.15 branch is cut. |
Fixes a crash in this case.