-
Notifications
You must be signed in to change notification settings - Fork 0
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 の改善 #4
Comments
Merged
そのキャッシュ方法だとダメだったみたいな話じゃなかったっけ?w |
ちゃんと動いてるからmatrix消し忘れてた |
Merged
a01sa01to
added a commit
that referenced
this issue
Jul 3, 2023
- checkout処理の重複を削除 - Cacheをsetup-nodeにお任せ - matrixを定義
#6 の checks を見てもらうと動いてるんですよねぇ
もし今後複数バージョンで動かす場合を考えて matrix を設定しておいた |
https://gist.github.com/belgattitude/838b2eba30c324f1f0033a797bab2e31?permalink_comment_id=4489311 |
使い方にもよるし、現時点では「Install deps」部分が 5s しかかかってないしそこまで大差ない気はするけどなぁ... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
lint.yml も同様っぽい...?
ui/.github/workflows/format.yml
Lines 10 to 14 in 548c9f4
ui/.github/workflows/format.yml
Line 15 in 548c9f4
cache: pnpm
を渡すと一瞬ui/.github/workflows/format.yml
Lines 29 to 47 in 548c9f4
The text was updated successfully, but these errors were encountered: