-
Notifications
You must be signed in to change notification settings - Fork 0
[chore]create deploy pipeline #3
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
Conversation
507e674 to
3ca5921
Compare
.github/workflows/deploy.yaml
Outdated
| - name: Set up JDK | ||
| uses: actions/setup-java@v4 | ||
| with: | ||
| java-version: '21' |
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.
@foo-543674 version21で大丈夫ですかね?
元々のflexmarkが使用していたものと合わせておいた方が、JVMのバージョン違いによる問題に合わないのでは?
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.
@raydive ありがとうございます。アプリのバージョンアップに合わせたほうがいいかなと思いましたが、問題が併発するのも避けたいので一旦下げました。
|
@foo-543674 |
corgimkii
left a comment
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.
LGTM
GitHub ActionsでCI/CDを構築いたしました。
テスト結果はChecksで確認できるようにしております。一部Fork前にすでにレッドになっていたテストがあったり、依存関係の解決でエラーになっていたので、そこだけ無視・修正しております。
デプロイはPEチームと調整して、OICDで認証を通すようにいたしました。
デプロイ方法に関しては一先ずデプロイ頻度もそこまで多くない見込みなので、ローカル作業を増やして自動実行するよりは、バージョンの書き換えまで自動化したうえで、実行を手動にして、手順忘れの防止を優先する方法といたしました。
バージョニングはupstreamとは明確に区別するために、"nu-"が接頭語としてつくようにしています。