Skip to content

Conversation

@t-miya
Copy link
Contributor

@t-miya t-miya commented Jan 14, 2026

26.2 に更新

@t-miya t-miya requested a review from zztkm January 14, 2026 05:53
Comment on lines 100 to 104
- [FIX] GitHub Actions のビルド環境を更新する
- GitHub Actions でのビルドが通らなくなったため
- Xcode の version を 16.4 に変更
- SDK を iOS 18.5 に変更
- @t-miya
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FIX 残しても良いと思いました。[対応はお任せ]

自分が FIX を残すとしたら、経緯がわかりやすいように PR の URL か Actions の履歴のリンクを貼って、「ビルドが通らなくなった」をより詳細に「macOS 15 利用中に error: iOS 18.4 Platform Not Installed. というエラー出るようになったため、SDK を iOS 18.5 に上げた」とか書くと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FIX の内容を戻しました。またエラーの内容を記載するように修正しました。

Copy link
Contributor

@zztkm zztkm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CHANGES の記載方針についてちょっと意見を書きましたが、基本的な対応としては問題ないと思うので、LGTM です 👍🏼

Copy link
Contributor

@zztkm zztkm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[FIX] 部分の CHANGES が残してあって、背景がわかりやすくなったと思います、LGTM です!

@t-miya t-miya merged commit 584acdd into develop Jan 14, 2026
1 check passed
@t-miya t-miya deleted the feature/actions-ios-version branch January 14, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants