From e6f769a812ecd2ab75ae8b8564d8afbbd5273a25 Mon Sep 17 00:00:00 2001 From: Thomas Stromberg Date: Wed, 5 Nov 2025 18:35:41 -0500 Subject: [PATCH] use :large_green_circle: for outgoing --- pkg/home/ui.go | 8 ++++---- pkg/home/ui_test.go | 16 ++++++++-------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/pkg/home/ui.go b/pkg/home/ui.go index 5fef24a..f6551af 100644 --- a/pkg/home/ui.go +++ b/pkg/home/ui.go @@ -140,9 +140,9 @@ func BuildPRSections(incoming, outgoing []PR) []slack.Block { var indicator string switch { case prs[i].NeedsReview: - indicator = ":red_circle:" + indicator = ":large_red_circle:" case prs[i].IsBlocked: - indicator = ":red_circle:" + indicator = ":large_red_circle:" case prs[i].ActionKind != "": indicator = ":speech_balloon:" default: @@ -207,9 +207,9 @@ func BuildPRSections(incoming, outgoing []PR) []slack.Block { var indicator string switch { case prs[i].NeedsReview: - indicator = ":green_circle:" + indicator = ":large_green_circle:" case prs[i].IsBlocked: - indicator = ":green_circle:" + indicator = ":large_green_circle:" case prs[i].ActionKind != "": indicator = ":speech_balloon:" default: diff --git a/pkg/home/ui_test.go b/pkg/home/ui_test.go index 7aa7351..083e9a4 100644 --- a/pkg/home/ui_test.go +++ b/pkg/home/ui_test.go @@ -143,13 +143,13 @@ func TestBuildBlocks(t *testing.T) { foundBlockedPR := false for _, block := range blocks { if sb, ok := block.(*slack.SectionBlock); ok { - if sb.Text != nil && strings.Contains(sb.Text.Text, ":red_circle:") { + if sb.Text != nil && strings.Contains(sb.Text.Text, ":large_red_circle:") { foundBlockedPR = true } } } if !foundBlockedPR { - t.Error("expected PR with :red_circle: indicating blocked incoming PR") + t.Error("expected PR with :large_red_circle: indicating blocked incoming PR") } }, }, @@ -195,13 +195,13 @@ func TestBuildBlocks(t *testing.T) { foundBlocked := false for _, block := range blocks { if sb, ok := block.(*slack.SectionBlock); ok { - if sb.Text != nil && strings.Contains(sb.Text.Text, ":green_circle:") { + if sb.Text != nil && strings.Contains(sb.Text.Text, ":large_green_circle:") { foundBlocked = true } } } if !foundBlocked { - t.Error("expected PR with :green_circle: for blocked outgoing PR") + t.Error("expected PR with :large_green_circle: for blocked outgoing PR") } }, }, @@ -410,10 +410,10 @@ func TestBuildPRSections_SortOrder(t *testing.T) { } // Verify color indicators - if !strings.Contains(incomingText, ":red_circle:") { - t.Error("incoming blocked PRs should use :red_circle:") + if !strings.Contains(incomingText, ":large_red_circle:") { + t.Error("incoming blocked PRs should use :large_red_circle:") } - if !strings.Contains(outgoingText, ":green_circle:") { - t.Error("outgoing blocked PRs should use :green_circle:") + if !strings.Contains(outgoingText, ":large_green_circle:") { + t.Error("outgoing blocked PRs should use :large_green_circle:") } }