Teach brokenImport String()

Author: zmackie

internal/feedback/feedback.go

@@ -88,6 +88,23 @@ type brokenImport struct {
 	revision    *gps.StringDiff
 }
 
+func (bi brokenImport) String() string {
+	var pr string
+	var cr string
+	if bi.revision != nil {
+		pr = fmt.Sprintf("(%s)", trimSHA(bi.revision.Previous))
+		cr = fmt.Sprintf("(%s)", trimSHA(bi.revision.Current))
+	}
+
+	return fmt.Sprintf("%v %s for %s. Locking in %v %s",
+		bi.version.Previous,
+		pr,
+		bi.projectPath,
+		bi.version.Current,
+		cr,
+	)
+}
+
 // ConstraintFeedback holds project constraint feedback data
 type BrokenImportFeedback struct {
 	brokenImports []brokenImport
@@ -108,13 +125,7 @@ func NewBrokenImportFeedback(ld *gps.LockDiff) *BrokenImportFeedback {
 
 func (b BrokenImportFeedback) LogFeedback(logger *log.Logger) {
 	for _, bi := range b.brokenImports {
-		logger.Printf("Warning: Unable to preserve imported lock %v (%v) for %v. Locking in %v (%v)\n",
-			bi.version.Previous,
-			trimSHA(bi.revision.Previous),
-			bi.projectPath,
-			bi.version.Current,
-			trimSHA(bi.revision.Current),
-		)
+		logger.Printf("Warning: Unable to preserve imported lock %v\n", bi)
 	}
 }