Skip to content

R8がexpo-modulesのクラスを勝手に削除しないように修正#5121

Merged
TinyKitten merged 1 commit intodevfrom
fix/background-tasks
Jan 24, 2026
Merged

R8がexpo-modulesのクラスを勝手に削除しないように修正#5121
TinyKitten merged 1 commit intodevfrom
fix/background-tasks

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Jan 24, 2026

Summary by CodeRabbit

  • その他
    • Android アプリのビルド最適化設定を更新しました。

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

@TinyKitten TinyKitten self-assigned this Jan 24, 2026
@TinyKitten TinyKitten changed the title R8がexpo-modulesののクラスを勝手に削除しないように修正 R8がexpo-modulesのクラスを勝手に削除しないように修正 Jan 24, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 24, 2026

📝 Walkthrough

概要

このプルリクエストはAndroidアプリのProGuard設定を修正し、React AnimatedおよびTurboModuleの既存の保持ルールを削除して、Expoモジュール向けの新しい保持ルールを追加しています。

変更内容

実装グループ / ファイル 変更内容
ProGuard設定ルール更新
android/app/proguard-rules.pro
com.swmansion.reanimated.** および com.facebook.react.turbomodule.** の保持ルールを削除。expo.modules.** の新規保持ルール({ *; })を追加。難読化時に保持されるクラスの対象を変更

見積もりコードレビュー負荷

🎯 1 (Trivial) | ⏱️ ~3 minutes

🐰✨
Expoモジュール、新しく守られ
古きルールは役目を終え
難読化の波も越えて
アプリケーション、さらに強く
前へ進もう!

🚥 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タイトルは変更内容と完全に関連しており、ProGuard/R8のルール変更によるexpo-modulesの保持についての主要な変更を明確に要約しています。
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 74ebb13 into dev Jan 24, 2026
4 checks passed
@TinyKitten TinyKitten deleted the fix/background-tasks branch January 24, 2026 02:58
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