-
Notifications
You must be signed in to change notification settings - Fork 41
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
CI 環境を構築する課題を作成 #40
CI 環境を構築する課題を作成 #40
Conversation
refs #35
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.
Session.0 のCIにReviewerを設定しようって項目を追加してもよいかも
GitHub Actions で PR が更新されたらビルドとテストを実行する CI 環境を構築してみよう | ||
|
||
## 参考 | ||
- [GitHub Actions で XCTest を実行する](https://zenn.dev/yumemi_inc/articles/xctest-github-actions) |
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.
今ってyumemi orgに入れてるんですっけ
publicにしてyumemi-inc or 個人にいれるとレビュワーの問題があるが
actionsでレビュワーの自動アサインはできたので、auto_request_review.ymlの設定さえ用意しちゃえばpublic運用にしちゃってもよいかも
https://github.com/marketplace/actions/auto-request-review
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.
yumemi org で private で実施し、終了後に public -> 個人のアカウントに transfer となっています。
この後、個人で private にして、 CI を回してしまって課金事故に繋がるケースもあり得そうなので、課金に関する注意事項も追記した方が良さそうですね。
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.
GitHub Actions のセルフホステッドランナーを利用した場合、課金事故は防げますが、
今度は、 private -> public にした場合、セキュリティホールになるリスクがあります。
いずれにせよ、 GitHub Actions を利用してもらうなら、そういった観点の理解も必要になり、それをこの研修の目的にすべきなのか検討する必要があります。
この部分が解消できないので一旦取り下げます。 |
Public で研修課題を行う場合は、課金について考慮は不要ですが、
Private の場合、 GitHub Actions の利用は金額的に現実的ではないかもしれないのが悩ましいです。
これに対しては、セルフホステッドランナーを利用する内容も含めました。
ただ、それにより Session0 の内容としては、重い気がするので悩み中です。
fixes #35