fix: show approve in green, not in the merged purple - #363
Merged
Merged
Conversation
approve の attachment に `Color::Merged` (`#6F42C1`) を使っていた。GitHub で merged PR を表す紫なので、merge していない PR の approve が merge されたように 見える。approve は GitHub でも緑なので、緑にする。 既存の `Color::Good` は流用しない。Slack のキーワード色で、issue / PR の opened や assigned に使っているため、approve が同じ色に埋もれる。GitHub の Approved と同じ success 緑 (`#1F883D`) を `Color::Approved` として持つ。 あわせて、面を塗る色として不自然な 2 点を直す。 コメント類の `#24292F` は Primer の fg.default で、文字色として定義された値。 Slack のダークテーマでは背景に溶けて色が付いていないように見えるので、 neutral (`#6E7781`) にする。 `Merged` / `Closed` の hex は Primer 更新前の旧パレットだったので、新しく入れる 緑と世代が揃うように現行の値 (`#8250DF` / `#CF222E`) に直す。どちらも今は 使っていないが、定義として残してある。 changes_requested の `danger` は変えていない。GitHub でも Changes requested は 赤で、こちらは不自然ではない。 hex を間違えても Slack は黙って受けるので、パレットの値そのものを固定する テストと、approve が緑で出ることのテストを足す。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
No unresolved issues remain, and the color changes have regression coverage.
Pull request overview
Updates Slack attachment colors so approved reviews use GitHub’s success green instead of merged purple.
Changes:
- Adds
Color::Approvedwith updated palette values. - Maps approved reviews to the new color.
- Adds palette and rendering regression tests.
File summaries
| File | Summary |
|---|---|
src/slack.rs |
Defines and tests the updated color palette. |
src/message.rs |
Uses approved green for approved reviews and tests rendering. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
approve の attachment が
Color::Merged(#6F42C1) だった。GitHub で merged PR を表す紫なので、merge していない PR の approve が merge されたように見える。Color::Goodは流用していない。Slack のキーワード色で issue / PR の opened・assigned に使っているため、approve が同じ緑に埋もれる。GitHub の Approved と同じ success 緑 (#1F883D) をColor::Approvedとして持たせた。#6F42C1(merged の紫)#1F883D(success の緑)#24292F(fg.default)#6E7781(neutral)Merged(未使用)#6F42C1#8250DFClosed(未使用)#CB2431#CF222Eあわせて、面を塗る色として不自然だった 2 点を直した。
#24292Fは Primer の fg.default で、文字色として定義された値。Slack のダークテーマでは背景に溶けて色が付いていないように見えるので neutral にしたMerged/Closedの hex は Primer 更新前の旧パレットだったので、新しく入れる緑と世代を揃えた。どちらも今は使っていないが、定義としては残してあるchanges_requestedのdangerは変えていない。GitHub でも Changes requested は赤なので、不自然ではないと判断した。hex を間違えても Slack は黙って受けるので、パレットの値そのものを固定するテストと、approve が緑で出ることのテストを足した。
🤖 Generated with Claude Code