Skip to content

canaryクラッシュ対策#5503

Merged
TinyKitten merged 1 commit intodevfrom
fix/canary-crash
Mar 1, 2026
Merged

canaryクラッシュ対策#5503
TinyKitten merged 1 commit intodevfrom
fix/canary-crash

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 1, 2026

overridesからhermes-compilerを消した

Summary by CodeRabbit

リリースノート

  • 設定変更

    • Hermesエンジンの不要な設定を削除し、ビルド構成を最適化しました。
    • 余分な依存関係オーバーライドをクリーンアップしました。
  • 新機能の準備

    • Firebase認証機能の統合基盤を追加しました。

@TinyKitten TinyKitten self-assigned this Mar 1, 2026
@github-actions github-actions bot added the ios label Mar 1, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 1, 2026

📝 Walkthrough

ウォークスルー

このPRはHermes V1の設定をiOS関連ファイルから削除し、React Nativeのビルド設定を再構成し、Expo設定にFirebase認証とアプリプラグインを追加します。複数の設定ファイルが更新されます。

変更内容

コホート / ファイル(s) 概要
Expo構成とiOSビルド設定
app.config.ts, ios/Podfile.properties.json
Hermes V1設定を削除し、ios.buildReactNativeFromSourceを適切にネストし、Firebaseアプリ認証プラグインを追加。
パッケージ依存関係オーバーライド
package.json
パッケージオーバーライドからhermes-compilerエントリを削除。

推定コードレビュー所要時間

🎯 2 (Simple) | ⏱️ ~10 分

関連する可能性のあるPR

提案されるラベル

ios

🐰 設定ファイルの中で、
Hermesを手放し、
Firebaseを迎え入れ、
iOSのビルドは新たな道へ。
兎のような素早さで、変更は完了!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PRタイトル「canaryクラッシュ対策」は、hermes-compilerを削除することでcanaryビルドのクラッシュに対応するという主要な変更内容を的確に要約しており、簡潔で明確です。
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 fix/canary-crash

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

@TinyKitten TinyKitten merged commit 3fd8ab1 into dev Mar 1, 2026
7 checks passed
@TinyKitten TinyKitten deleted the fix/canary-crash branch March 1, 2026 05:03
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