-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: buildConfig に flavor を追加 #290
Conversation
Walkthroughこの変更は、AndroidとiOSのアプリケーションにおいて、開発( Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (9)
- app/android/src/androidMain/kotlin/club/nito/app/AppModule.kt (2 hunks)
- app/ios/App/Nito/Dev/Info.plist (1 hunks)
- app/ios/App/Nito/Nito.xcodeproj/project.pbxproj (7 hunks)
- app/ios/App/Nito/Nito.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved (2 hunks)
- app/ios/App/Nito/Prod/Info.plist (1 hunks)
- app/ios/Modules/Package.resolved (2 hunks)
- app/ios/Modules/Sources/KmpContainer/Container.swift (2 hunks)
- core/model/src/commonMain/kotlin/club/nito/core/model/BuildConfig.kt (2 hunks)
- core/model/src/commonMain/kotlin/club/nito/core/model/Flavor.kt (1 hunks)
Files skipped from review due to trivial changes (1)
- core/model/src/commonMain/kotlin/club/nito/core/model/Flavor.kt
Additional comments: 18
app/ios/App/Nito/Dev/Info.plist (1)
- 5-6:
Info.plist
にFlavor
キーとその値としてdev
を追加することで、開発ビルドを識別できるようになりました。この変更は、アプリケーションが開発環境で実行されているかどうかをプログラムで判断するために使用できます。app/ios/App/Nito/Prod/Info.plist (1)
- 5-6:
Info.plist
にFlavor
キーとその値としてprod
を追加することで、本番ビルドを識別できるようになりました。この変更は、アプリケーションが本番環境で実行されているかどうかをプログラムで判断するために使用できます。app/android/src/androidMain/kotlin/club/nito/app/AppModule.kt (2)
- 4-4:
Flavor
enumをclub.nito.core.model
からインポートしました。これにより、アプリケーションのビルドフレーバーに基づいて異なる動作を実装できるようになります。- 14-18:
AppBuildConfig.FLAVOR
プロパティに基づいて、flavor
プロパティの値を設定しています。このロジックにより、ビルドフレーバー(開発または本番)に基づいてアプリケーションの設定を動的に変更できます。デフォルトのフレーバーとしてFlavor.Dev
を使用することは、未知のフレーバー値が設定された場合の安全なフォールバックとして機能します。core/model/src/commonMain/kotlin/club/nito/core/model/BuildConfig.kt (2)
- 22-22:
BuildConfig
インターフェースに新しい公開プロパティflavor
を追加しました。これにより、アプリケーションのビルドフレーバーを表すことができるようになります。- 37-37:
EmptyBuildConfig
クラスにflavor
プロパティの実装を追加し、デフォルト値としてFlavor.Dev
を設定しました。これは、ビルド設定が明示的に提供されない場合に、開発フレーバーをデフォルトとする合理的な選択です。app/ios/Modules/Sources/KmpContainer/Container.swift (2)
- 28-28:
AppBuildConfig
クラスに新しいプロパティflavor
を追加しました。これは、iOSアプリケーションがビルドフレーバーを識別できるようにするための重要な変更です。- 41-48:
Info.plist
ファイルからFlavor
キーの値を読み取り、それに基づいてflavor
プロパティを初期化しています。このアプローチにより、アプリケーションのビルドフレーバーを動的に設定でき、開発と本番の環境を区別できます。app/ios/Modules/Package.resolved (2)
- 35-36:
swift-dependencies
のリビジョンとバージョンを更新しました。これは、プロジェクトの依存関係を最新の状態に保つための重要な変更です。- 53-54:
xctest-dynamic-overlay
のリビジョンとバージョンを更新しました。テストフレームワークの依存関係を最新に保つことは、安定したテスト環境を維持する上で重要です。app/ios/App/Nito/Nito.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved (3)
- 17-18:
swift-clocks
のリビジョンとバージョンを更新しました。依存関係の更新は、プロジェクトが最新のライブラリ機能を利用できるようにするために重要です。- 26-27:
swift-concurrency-extras
のリビジョンとバージョンを更新しました。非同期プログラミングをサポートするライブラリの最新バージョンを使用することは、アプリケーションのパフォーマンスと安定性に貢献します。- 44-45:
swift-syntax
のリビジョンとバージョンを更新しました。Swiftの構文解析を扱うライブラリの更新は、コード生成や解析機能の改善につながります。app/ios/App/Nito/Nito.xcodeproj/project.pbxproj (5)
- 30-31:
Info.plist
ファイルの参照を2つ追加しました。これにより、Xcodeプロジェクトが新しいビルドフレーバーに対応するInfo.plist
ファイルを認識できるようになります。- 77-77:
Prod
ビルド設定のINFOPLIST_FILE
パスを更新しました。これにより、本番ビルド用の正しいInfo.plist
ファイルが参照されるようになります。- 103-103:
Dev
ビルド設定のINFOPLIST_FILE
パスを更新しました。これにより、開発ビルド用の正しいInfo.plist
ファイルが参照されるようになります。- 383-383:
Prod
ターゲットのINFOPLIST_FILE
設定をProd/Info.plist
に更新しました。これは、本番ビルドの設定を正しく指定するための重要な変更です。- 452-452:
Dev
ターゲットのINFOPLIST_FILE
設定をDev/Info.plist
に更新しました。これは、開発ビルドの設定を正しく指定するための重要な変更です。
Issue
概要
buildConfig に flavor を追加します.
レビュー観点
レビューレベル
レビュー優先度
参考リンク
スクリーンショット
Summary by CodeRabbit