-
Notifications
You must be signed in to change notification settings - Fork 205
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
Add review badge when all jobs passed or reviewed #1064
Conversation
Implement optional task in https://progress.opensuse.org/issues/14934
@@ -68,6 +68,8 @@ sub add_review_badge { | |||
my ($build_res) = @_; | |||
|
|||
$build_res->{reviewed_all_passed} = $build_res->{passed} == $build_res->{total}; | |||
$build_res->{reviewed_all_passed_or_softfailed} | |||
= $build_res->{passed} + $build_res->{softfailed} == $build_res->{total}; |
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.
this reviewed_ prefix is actually wrong. There is nothing about reviews in 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.
Think about it as "auto-review" badge. But I can change it if you like. Then I would also change reviewed_all_passed
for consistency.
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.
I disagree that passed tests are auto-reviewed. They are not reviewed at all
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.
Changed it
fine with me |
Implement optional task in https://progress.opensuse.org/issues/14934