Skip to content
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

[思想バトル]delete bun as a package manager #203

Merged
merged 3 commits into from
Nov 9, 2023

Conversation

Azuki-bar
Copy link
Member

@Azuki-bar Azuki-bar commented Nov 8, 2023

このPRは思想が強く出ているものなので、もし余計なお世話だと思ったらCloseしてください。もしくはリプライで議論しましょう。

bunをパッケージマネージャとして使うと、bunはロックファイルとしてバイナリを生成する。

しかしバイナリはGitHubのdiffビューワーで見ることができない。そのためどのようなライブラリが同時にインストールされたか確認することが難しい。
一応bunはロックファイルとしてyarn形式のロックファイルを同時に吐き出すことができるが、例え同じ生成元のロックファイルと言えど複数存在しているのはどちらを信用してよいか分からなくなる。

pnpmはhuman readableなロックファイルを生成する。また依存内で共通したパッケージはシンボリックリンクを使うためインストールがnpmと比べて速い。

欠点はタイピングが難しい。

@github-actions github-actions bot added the video-sender カメラ映像関連 label Nov 8, 2023
@Azuki-bar
Copy link
Member Author

https://pnpm.io/workspaces を使っている。

workspacesを使うことにより、ルートディレクトリでpnpm iを実行するとルートディレクトリにnode_modulesが生成され、そして全てのworkspaces以下に対してnpm i相当のコマンドが実行される。
各workspacesの下にもnode_modulesが生成され、これらの中に格納されるモジュールはルートのnode_modulesの中へとリンクされたシンボリックリンクになっている。

@csenet csenet self-requested a review November 9, 2023 02:51
Copy link
Member

@csenet csenet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bunにそこまで強いこだわりがあるわけではないので、pnpm使ってみるのもありだと思います
確かにlockファイルがhuman readableなのは良い気がします

@Azuki-bar
Copy link
Member Author

いきます
ありがとうございます

@Azuki-bar Azuki-bar merged commit 6ec1fe9 into main Nov 9, 2023
3 checks passed
@Azuki-bar Azuki-bar deleted the migrate-bun-to-pnpm branch November 9, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
video-sender カメラ映像関連
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants