From 958b56511b5bb3ce4658a594d63065208d8d5323 Mon Sep 17 00:00:00 2001 From: BakerNet Date: Thu, 3 Jul 2025 13:04:38 -0700 Subject: [PATCH] Update comment even if all approved --- internal/app/app.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/app/app.go b/internal/app/app.go index 7856e02..c488338 100644 --- a/internal/app/app.go +++ b/internal/app/app.go @@ -294,7 +294,7 @@ func (a *App) processApprovalsAndReviewers() (bool, string, []string, error) { func (a *App) addReviewStatusComment(allRequiredOwners codeowners.ReviewerGroups, maxReviewsMet bool) error { // Comment on the PR with the codeowner teams required for review - if a.config.Quiet || len(allRequiredOwners) == 0 { + if a.config.Quiet { a.printDebug("Skipping review status comment (disabled or no unapproved owners).\n") return nil } @@ -334,7 +334,7 @@ func (a *App) addReviewStatusComment(allRequiredOwners codeowners.ReviewerGroups if err != nil { return fmt.Errorf("UpdateComment Error: %v", err) } - } else { + } else if len(allRequiredOwners) > 0 { a.printDebug("Adding new review status comment: %q\n", comment) err = a.client.AddComment(comment) if err != nil {