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

Yarn を pnpm に移行する #250

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Yarn を pnpm に移行する #250

wants to merge 23 commits into from

Conversation

fsubal
Copy link
Member

@fsubal fsubal commented Mar 1, 2023

やったこと

ref: #240

Yarn の PnP モードで得られる「本来追加すべき dependencies のチェックが厳密になる」というメリットは pnpm でも得られる。yarn と違ってエディタ向けの特殊対応とかも必要がない。

そして、pnpm の方が OSS で見かけるのでこっちにしたらどうなるか検証したい( next.js のリポジトリも pnpm を使っている )

動作確認環境

CI が通ること、手元でエディタが普通に動くこと、Storybook が起動すること

チェックリスト

不要なチェック項目は消して構いません

  • 破壊的変更がある場合には、対象のパッケージのメジャーバージョンが上がっていることを確認した
  • 追加したコンポーネントが index.ts から再 export されている
  • README やドキュメントに影響があることを確認した

@fsubal fsubal marked this pull request as ready for review March 1, 2023 14:51
@fsubal fsubal marked this pull request as draft March 1, 2023 14:52
@fsubal fsubal marked this pull request as ready for review March 23, 2023 07:41
@fsubal fsubal force-pushed the use-pnpm branch 2 times, most recently from 89a4c9c to 789c822 Compare March 28, 2023 15:56
@fsubal fsubal changed the title (検証)Yarn を捨てて pnpm にしてみる Yarn を pnpm に移行する Mar 28, 2023
@fsubal fsubal added the minor label Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants