-
Notifications
You must be signed in to change notification settings - Fork 309
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
Improve: テストを高速化 #1302
Improve: テストを高速化 #1302
Conversation
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.
分けるの良いですね!
提案されている大多数部分はコピペコードになっていて、メンテナンス性が下がりそうです。
共通部分をActionsに切り出すか、あるいはCache部分をすべて消してしまうのはどうでしょう。
個人的にはキャッシュは結構危ういので後者のほうが良いかなと思っています。
Action切り出しするならついでに他でNodeインストールとかしている部分も変えちゃいたいですね!
環境構築周りを共通化しました。 |
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!!
同時実行数が増えてきたら逆に遅くなることもありえるかもとちょっと思いました。
内容
テストのWorkflowを高速化します。
関連 Issue
(なし)
スクリーンショット・動画など
(なし)
その他
(なし)