diff --git a/pkg/diff/ai_summary.go b/pkg/diff/ai_summary.go index fa31a2dc..760e150d 100644 --- a/pkg/diff/ai_summary.go +++ b/pkg/diff/ai_summary.go @@ -23,7 +23,7 @@ func AIDiffSummary(ctx context.Context, mrNote *pkg.Message, name string, manife if err != nil { telemetry.SetError(span, err, "OpenAI SummarizeDiff") log.Error().Err(err).Msg("failed to summarize diff") - cr := pkg.CheckResult{State: pkg.StateWarning, Summary: "failed to summarize diff", Details: err.Error()} + cr := pkg.CheckResult{State: pkg.StateNone, Summary: "failed to summarize diff", Details: err.Error()} mrNote.AddToAppMessage(ctx, name, cr) return } diff --git a/pkg/vcs/github_client/client.go b/pkg/vcs/github_client/client.go index f7c9ebef..d66c8e63 100644 --- a/pkg/vcs/github_client/client.go +++ b/pkg/vcs/github_client/client.go @@ -156,17 +156,16 @@ func toGithubCommitStatus(state pkg.CommitState) *string { switch state { case pkg.StateError, pkg.StatePanic: return pkg.Pointer("error") - case pkg.StateFailure, pkg.StateWarning: + case pkg.StateFailure: return pkg.Pointer("failure") case pkg.StateRunning: return pkg.Pointer("pending") - case pkg.StateSuccess: + case pkg.StateSuccess, pkg.StateWarning, pkg.StateNone: return pkg.Pointer("success") - - default: // maybe a different one? panic? - log.Warn().Str("state", state.String()).Msg("failed to convert to a github commit status") - return pkg.Pointer("failure") } + + log.Warn().Str("state", state.String()).Msg("failed to convert to a github commit status") + return pkg.Pointer("failure") } func (c *Client) CommitStatus(ctx context.Context, repo *repo.Repo, status pkg.CommitState) error {