AI summary does not fail the build (#103)

* AI summary does not fail the build

* clean up github statuses

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
 	}

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 {