From a785bcdad6296b3babeb62ef3e91d66450730244 Mon Sep 17 00:00:00 2001
From: Josh Bialkowski <josh.bialkowski@gmail.com>
Date: Mon, 16 Dec 2019 09:09:42 -0800
Subject: codereview #6: don't fail one warning

* presence of an error in the import event doesn't indicate failure
---
 bridge/core/import.go | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

(limited to 'bridge/core/import.go')

diff --git a/bridge/core/import.go b/bridge/core/import.go
index 5f138da7..f0a6f0c8 100644
--- a/bridge/core/import.go
+++ b/bridge/core/import.go
@@ -2,6 +2,7 @@ package core
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/MichaelMure/git-bug/entity"
 )
@@ -74,10 +75,18 @@ func (er ImportResult) String() string {
 		}
 		return fmt.Sprintf("import error: %s", er.Err.Error())
 	case ImportEventWarning:
+		parts := make([]string, 0, 4)
+		parts = append(parts, "warning:")
 		if er.ID != "" {
-			return fmt.Sprintf("warning at id %s: %s", er.ID, er.Err.Error())
+			parts = append(parts, fmt.Sprintf("at id %s", er.ID))
 		}
-		return fmt.Sprintf("warning: %s", er.Err.Error())
+		if er.Reason != "" {
+			parts = append(parts, fmt.Sprintf("reason: %s", er.Reason))
+		}
+		if er.Err != nil {
+			parts = append(parts, fmt.Sprintf("err: %s", er.Err))
+		}
+		return strings.Join(parts, " ")
 
 	default:
 		panic("unknown import result")
-- 
cgit