Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

GitHub Actionsのセットアップ(iOS) #147

Merged
merged 3 commits into from
May 11, 2021

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented May 2, 2021

Issue 番号 / Issue ID

目的 / Purpose

Pull Requestなどで、ファイルのアップロード漏れや特定のConfigurationでビルドできなくなるのを防ぐためにCIでビルドチェックを行いたい。

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

確認事項 / What to check

GitHub Actionsのビルドが成功するかを確認する。
https://github.com/cocoa-mhlw/cocoa/runs/2486259760

その他 / Other information

Debug|iPhoneSimulatorに設定されていたCodesignEntitlementsは必要ないと言う結論に至った(iPhoneSimulatorでは実際のExposure Notificationを使う必要はない)。

また、iPhone実機で動かすケース(Debug|iPhone, Release|iPhone, Adhoc|iPhoneなど)は、ExposureNotificationを利用可能なプロビジョニングプロファイルを設定する必要がある。これらのプロファイルは非公開かつ開発チームは個別で設定しているのでオープンソースのビルドはiPhoneSimulatorを前提にするのが適切と考える。

Copy link
Collaborator

@halsk halsk left a comment

Choose a reason for hiding this comment

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

LGTMです!ありがとうございます

@keiji keiji merged commit 7038d42 into cocoa-mhlw:master May 11, 2021
@keiji keiji deleted the setup_ci_ios branch August 19, 2021 03:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants