From 2a2f0546abd53c701f7e72a8bee44685dc2987e5 Mon Sep 17 00:00:00 2001 From: Harry Lawrence Date: Wed, 23 Dec 2015 16:18:22 +0000 Subject: [PATCH] Possible bug fix for thread comment status As pointed out in #22, the status for every comment seems to be fyi. This seems to fix that. --- commands/output/output.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/commands/output/output.go b/commands/output/output.go index 96e8c634..2ffa74c8 100644 --- a/commands/output/output.go +++ b/commands/output/output.go @@ -119,15 +119,16 @@ func showThread(r *review.Review, thread review.CommentThread) error { // showSubThread prints the given comment (sub)thread, indented by the given prefix string. func showSubThread(r *review.Review, thread review.CommentThread, indent string) error { + comment := thread.Comment statusString := "fyi" - if thread.Resolved != nil { - if *thread.Resolved { + if comment.Resolved != nil { + if *comment.Resolved { statusString = "lgtm" } else { statusString = "needs work" } } - comment := thread.Comment + threadHash, err := comment.Hash() if err != nil { return err