-
Notifications
You must be signed in to change notification settings - Fork 2
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
Move NodeJS package manager to pnpm #353
Conversation
b9765ce
to
83836cf
Compare
rebase |
bf67b10
to
23126ce
Compare
とのことですけども……(初めて見るエラーだ) |
あーこれ既に actions/setup-node が pnpm 入れてるから pnpm のインストールが衝突してるのか? |
rebase |
23126ce
to
08c2224
Compare
いや、 |
8a63434
to
9702b4f
Compare
やや不本意だが |
5b06a9f
to
0b70701
Compare
Co-authored-by: KOBA789 <koba789@arkedgespace.com>
0b70701
to
c92a8b5
Compare
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.
そういえば、pnpm
が打ちづらいのは公式も認識しているようで、インストール手順の最後にエイリアスを貼る方法が紹介されているほどです。
もしかしたら C2A のドキュメントでもそれを案内してもいい、かもしれない(要らんことは案内しないほうがいいという説もかなりあるけど)
https://pnpm.io/installation#using-a-shorter-alias
要らんことは案内しない方がいい(分かっている人は調べりゃ勝手にやるし、分かってない人が alias をゴチャッとやると悲惨だしデバッグが困難になる) |
そもそも打つ頻度がそんなにだし |
概要
SSIA
Issue
詳細
検証結果
pytest CI が通ればよし
影響範囲
C2A Boom