Skip to content

cleanup(ui): move log-level color comment onto the info branch - #1327

Merged
9thLevelSoftware merged 1 commit into
mainfrom
cursor/ui-log-level-comment-6cd7
Oct 23, 2026
Merged

9thLevelSoftware merged 1 commit into
mainfrom
cursor/ui-log-level-comment-6cd7

Conversation

@9thLevelSoftware

Copy link
Copy Markdown
Owner

A formatter split the trailing note off LogLevel.INFO in getLevelColor, so // Blue (informational, not semantic status) sat on the warning arm. The note describes colors.info (informational blue, not a semantic status). This moves that comment onto the info branch. No behavior change.

Open in Web Open in Cursor 

A formatter split the trailing note off LogLevel.INFO, so it sat on the warning arm. The note describes the informational blue, which is colors.info.

Co-authored-by: Devil <9thLevelSoftware@users.noreply.github.com>
@9thLevelSoftware
9thLevelSoftware marked this pull request as ready for review October 23, 2026 14:31
Copilot AI balanced review requested due to automatic review settings October 23, 2026 14:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kilo-code-bot

kilo-code-bot Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • shared/src/commonMain/kotlin/com/devil/phoenixproject/presentation/screen/ConnectionLogsScreen.kt

Ponytail: Lean already. Ship. Ponytail net: 0 lines.


Reviewed by minimax-m3 · Input: 28.6K · Output: 3.4K · Cached: 232.5K

Review guidance: REVIEW.md from base branch main

@9thLevelSoftware
9thLevelSoftware merged commit e02835b into main Oct 23, 2026
11 checks passed
@9thLevelSoftware
9thLevelSoftware deleted the cursor/ui-log-level-comment-6cd7 branch October 23, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants