Skip to content

Android Live UpdateのshortTextで「まもなく」を廃止し「次は」に統合#5417

Merged
TinyKitten merged 1 commit intodevfrom
claude/update-android-live-text-lXWtx
Feb 22, 2026
Merged

Android Live UpdateのshortTextで「まもなく」を廃止し「次は」に統合#5417
TinyKitten merged 1 commit intodevfrom
claude/update-android-live-text-lXWtx

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 22, 2026

https://claude.ai/code/session_01KBPf4rJnxLCCkDr1QvPGdx
文言が長すぎて表示しきれないので

Summary by CodeRabbit

バグ修正

  • 駅接近時のテキスト表示ロジックを修正しました。

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

📝 Walkthrough

Walkthrough

LiveUpdateModule.kt の shortCriticalText 計算から「接近中」状態の専用処理が削除され、次の駅名を使用する場合のロジック制御が変更されました。

Changes

コホート / ファイル 変更内容
接近状態処理の削除
android/app/src/main/java/me/tinykitten/trainlcd/LiveUpdateModule.kt
shortCriticalText 計算から「approaching」状態の専用ハンドリングを削除。nextStationName を使用する際の処理が else ブランチにフォールバックするよう変更。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

android

Poem

🐰 一行の削除で新たな流れ
approaching から next へと
駅名の表示が変わりゆく
シンプルな変更に思いを馳せ
コードの物語が進み行く ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PRのタイトルは、Android Live UpdateのshortTextで「まもなく」ラベルを廃止し「次は」に統合するという変更内容を正確かつ明確に説明しており、変更セットの主要な目的を適切に要約しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/update-android-live-text-lXWtx

Comment @coderabbitai help to get the list of available commands and usage tips.

@TinyKitten TinyKitten merged commit 19dce32 into dev Feb 22, 2026
4 of 5 checks passed
@TinyKitten TinyKitten deleted the claude/update-android-live-text-lXWtx branch February 22, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants