Skip to content

expo.modules.location.services.LocationTaskServiceをreplaceではなくmergeに変更#5148

Merged
TinyKitten merged 1 commit intodevfrom
fix/android-bg-tts
Jan 25, 2026
Merged

expo.modules.location.services.LocationTaskServiceをreplaceではなくmergeに変更#5148
TinyKitten merged 1 commit intodevfrom
fix/android-bg-tts

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Jan 25, 2026

#5133

Summary by CodeRabbit

  • その他の変更
    • 位置情報関連機能の内部設定を最適化しました。複数の設定が組み合わされる場合の属性処理方法を改善し、より柔軟で安定したサービス動作を実現しました。

✏️ Tip: You can customize this high-level summary in your review settings.

@TinyKitten TinyKitten self-assigned this Jan 25, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 25, 2026

📝 Walkthrough

Walkthrough

LocationTaskServiceのAndroidManifest.xmlの宣言を変更し、android:exported="false"属性を削除し、マニフェストマージ戦略をtools:node="replace"からtools:node="merge"に変更しました。

Changes

Cohort / File(s) 変更内容
Android Manifest Configuration
android/app/src/main/AndroidManifest.xml
LocationTaskServiceからandroid:exported="false"属性を削除し、マニフェストマージ戦略をreplaceからmergeに変更

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

android

Poem

🐰 Manifest の書き直し、すすすっと行き
マージ戦略を切り替えて、スムーズに
Androidの調和、今ここに実現
ぴょんぴょん

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは変更の主要な内容を正確に反映しており、LocationTaskServiceのマニフェスト戦略を「replace」から「merge」に変更したことを明確に説明しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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

@TinyKitten TinyKitten merged commit a810228 into dev Jan 25, 2026
4 checks passed
@TinyKitten TinyKitten deleted the fix/android-bg-tts branch January 25, 2026 04:26
@TinyKitten TinyKitten mentioned this pull request Jan 25, 2026
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.

1 participant