Skip to content
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

👍 Xcode プロジェクトの設定を変更 #107

Merged
merged 2 commits into from
Dec 3, 2023

Conversation

tatsutakein
Copy link
Member

@tatsutakein tatsutakein commented Dec 3, 2023

Issue

  • close #ISSUE_NUMBER 🦕

概要

Xcode プロジェクトの設定を変更します。

  • developmentRegion を ja に変更
  • IPHONEOS_DEPLOYMENT_TARGET を 17 に設定

レビュー観点

特になし

レビューレベル

  • Lv0: まったく見ないで Approve する
  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

参考リンク

スクリーンショット

Before After

Summary by CodeRabbit

  • 新機能

    • 開発地域を「英語(en)」から「日本語(ja)」に変更しました。
    • サポートされる地域に「日本語(ja)」を追加し、「英語(en)」を削除しました。
  • 改善点

    • iOSの最小サポートバージョンを16.0から17.0に更新しました。

@tatsutakein tatsutakein requested a review from a team as a code owner December 3, 2023 00:41
Copy link

coderabbitai bot commented Dec 3, 2023

Walkthrough

Xcodeプロジェクトの設定が変更され、開発地域が「en」から「ja」に変更されました。これにより、プロジェクトの既知の地域リストも更新され、「en」が除外され、「ja」が追加されています。また、iOSの最小サポートバージョンが16.0から17.0に上がっています。

Changes

ファイル 変更内容
.../Nito.xcodeproj/project.pbxproj developmentRegionを「en」から「ja」に変更、knownRegionsに「ja」を追加し「en」を除外、IPHONEOS_DEPLOYMENT_TARGETを16.0から17.0に更新

🐰
新しい地域、新しい始まり、
コードの森で跳ねる小さな夢、
iOSの世界が広がる時、
うさぎも一緒に飛び跳ねる。 🌟


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • If you reply to a review comment from CodeRabbit, the bot will automatically respond.
  • To engage with CodeRabbit bot directly around the specific lines of code in the PR, mention @coderabbitai in your review comment
  • Note: Review comments are made on code diffs or files, not on the PR overview.
  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai help to get help.

Note: For conversation with the bot, please use the review comments on code diffs or files.

CodeRabbit Configration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • The JSON schema for the configuration file is available here.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

Copy link

@coderabbitai coderabbitai bot left a 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

Commits Files that changed from the base of the PR and between fc4551f and 74a772d.
Files selected for processing (1)
  • app/ios/App/Nito/Nito.xcodeproj/project.pbxproj (3 hunks)
Additional comments: 3
app/ios/App/Nito/Nito.xcodeproj/project.pbxproj (3)
  • 175-183: 開発地域が "en" から "ja" に変更され、knownRegions に "ja" が追加されています。これは、プロジェクトのローカライゼーションと開発の焦点が日本に移行したことを示しています。変更は意図したとおりで、問題は見当たりません。

  • 287-293: IPHONEOS_DEPLOYMENT_TARGET が 16.0 から 17.0 に更新されています。これにより、アプリのサポートする最小 iOS バージョンが変更され、iOS 16.0 以前を実行しているデバイスではもはや互換性がなくなります。変更は意図したとおりで、問題は見当たりません。

  • 344-350: リリースビルド設定でも IPHONEOS_DEPLOYMENT_TARGET が 17.0 に設定されています。これはデバッグビルド設定と一致しており、一貫性が保たれています。変更は意図したとおりで、問題は見当たりません。

@tatsutakein tatsutakein merged commit d0326c0 into main Dec 3, 2023
4 checks passed
@tatsutakein tatsutakein deleted the rt/improve-ios-project branch December 3, 2023 05:23
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