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 に変更 #702

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

yarn から pnpm に変更 #702

wants to merge 9 commits into from

Conversation

naporin0624
Copy link
Member

@naporin0624 naporin0624 commented Feb 20, 2025

やったこと

動作確認環境

  • GitHub Actions
  • storybook
  • pnpm test
  • pnpm lint
  • pnpm fmt
  • pnpm build
  • pnpm typecheck
  • pnpm typecheck:config

チェックリスト

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

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

Copy link

github-actions bot commented Feb 20, 2025

Visit the preview URL for this PR (updated for commit 491e4a7):

https://pixiv-charcoal-web--pr702-naporitan-use-pnpm-0ek36ks1.web.app

(expires Thu, 27 Feb 2025 08:14:02 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 314b26d3adca98a761c7e4d9922ebb206ff024a0

Copy link
Contributor

@yue4u yue4u left a comment

Choose a reason for hiding this comment

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

ビルド&テスト&storybook動作良さそう。

#701 とコンフリクトするのでどっち先にマージするか決める必要があります。
また、ビルド構成が変わるので、一回betaとかとしてリリースしてみても良さそう

@@ -9,9 +9,12 @@ jobs:

steps:
- uses: actions/checkout@v3
- uses: pnpm/action-setup@v4
Copy link
Contributor

Choose a reason for hiding this comment

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

[QUESTION] setup-nodeしているとしてない箇所があると思いますがどういう使い分けですか?

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