From 88d7fb66a980ca729c60f1c4637c9e4f96ff9572 Mon Sep 17 00:00:00 2001 From: Kurosawa Takuma Date: Thu, 9 Nov 2023 19:30:43 +0700 Subject: [PATCH] Send error comment before plan preview exit (#4658) * Send error comment before plan preview exit Signed-off-by: Takumaron * Apply review commnet Signed-off-by: Takumaron --------- Signed-off-by: Takumaron --- tool/actions-plan-preview/main.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tool/actions-plan-preview/main.go b/tool/actions-plan-preview/main.go index 9194030543..0bf0ecdcc3 100644 --- a/tool/actions-plan-preview/main.go +++ b/tool/actions-plan-preview/main.go @@ -101,14 +101,14 @@ func main() { } if event.PRClosed { - doComment(failureBadgeURL + "Unable to run plan-preview for a closed pull request.") + doComment(failureBadgeURL + "\nUnable to run plan-preview for a closed pull request.") return } // TODO: When PR opened, `Mergeable` is nil for calculation. // Here it is not considered for now, but needs to be handled. if event.PRMergeable != nil && *event.PRMergeable == false { - doComment(failureBadgeURL + "Unable to run plan-preview for an un-mergeable pull request. Please resolve the conficts and try again.") + doComment(failureBadgeURL + "\nUnable to run plan-preview for an un-mergeable pull request. Please resolve the conficts and try again.") return } @@ -123,6 +123,7 @@ func main() { args.Timeout, ) if err != nil { + doComment(failureBadgeURL + "\nUnable to run plan-preview. \ncause: " + err.Error()) log.Fatal(err) } log.Println("Successfully retrieved plan-preview result") @@ -131,10 +132,11 @@ func main() { if result.HasError() { pr, err := getPullRequest(ctx, ghClient.PullRequests, event.Owner, event.Repo, event.PRNumber) if err != nil { + doComment(failureBadgeURL + "\nUnable to run plan-preview. \ncause: " + err.Error()) log.Fatal(err) } if !pr.GetClosedAt().IsZero() { - doComment(failureBadgeURL + "Unable to run plan-preview for a closed pull request.") + doComment(failureBadgeURL + "\nUnable to run plan-preview for a closed pull request.") return } }