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

chore: pnpmをv10に更新 #2528

Merged
merged 4 commits into from
Feb 12, 2025

Conversation

sevenc-nanashi
Copy link
Member

内容

pnpmを10.3に更新します。

関連 Issue

(なし)

スクリーンショット・動画など

(なし)

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner February 12, 2025 01:00
@sevenc-nanashi sevenc-nanashi requested review from Hiroshiba and removed request for a team February 12, 2025 01:00
@sevenc-nanashi
Copy link
Member Author

ふと:scriptsにこういう区切り線を入れても良いのかも...?
https://github.com/samchon/typia/blob/e0a7ffae5d1e04a4995591bf010348a7eee8568f/package.json#L20

@Hiroshiba
Copy link
Member

ふと:scriptsにこういう区切り線を入れても良いのかも...?

jsonフォーマッターが勝手に空行を消してくるから、ですよね!
(経験上、後で意図がわからなくなるのでwhatだけじゃなくwhyも書く癖を付けておくと便利)

まあありなのかなと思いました!
同じキーだとjson的によろしくなさそうなのと、あとコメントであることが直感的にわかるように先頭に// を付けても良いかも。

"// --- tests ---": ""
"// --- scripts ---": ""
みたいな!

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

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

LGTM!!

nodeのバージョン更新もちょうどあったので、一緒に合わせて案内issueを作ってpin留めしておこうと思います!

ちなみにpnpm 10の利点はなんでしょう?issueで案内したく。

@sevenc-nanashi
Copy link
Member Author

sevenc-nanashi commented Feb 12, 2025

pnpm 10の利点はなんでしょう?

リリースノート(https://github.com/pnpm/pnpm/releases/v10.0.0)をざっくり抜粋します:

  • セキュリティ向上
    • postinstallのホワイトリスト化(賛否両論ではあるけど)
    • node_modules内のsha256が録られるようになったらしい
  • manage-package-manager-versionsがオンになった
  • verify-deps-before-runが増えた(pnpm run時にnode_modulesが最新かどうかをチェックできる)
  • pnpm addがcatalog(Cargoのworkspace.dependencies相当)を見るようになった

あとpnpmの案内は特に要らないと想います、というのもpnpm iすればpackageManagerを読んで自動でv10が入ってくれるので(実際自分がバージョンを上げるときもpackageManagerの値を変えてpnpm iするだけで行った)

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 12, 2025

ありがとうございます!
まあ確かに案内は要らないかもですね。node_modulesの全ダウンロードを走らせないとだけど、まあ別に良いか!

@Hiroshiba Hiroshiba enabled auto-merge February 12, 2025 02:10
@Hiroshiba Hiroshiba added this pull request to the merge queue Feb 12, 2025
@sevenc-nanashi sevenc-nanashi removed this pull request from the merge queue due to a manual request Feb 12, 2025
@sevenc-nanashi sevenc-nanashi added this pull request to the merge queue Feb 12, 2025
github-merge-queue bot pushed a commit that referenced this pull request Feb 12, 2025
Merged via the queue into VOICEVOX:main with commit f44a29b Feb 12, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants