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: upgrade pnpm to >=9 #500

Merged
merged 2 commits into from
Aug 13, 2024

Conversation

senkenn
Copy link
Contributor

@senkenn senkenn commented Jul 20, 2024

📑 Summary

  • pnpm が v9 になったことで lockfile のフォーマットが大きく変わり、pnpm i するだけで lockfile に変更が入ってしまうのでバージョンを上げたほうがいいと思いました。
    image

  • 2024/7/21時点での latest version: v9.5.0 で更新しました。

📋 Task

  • 📖 Contribution Guide を読んだ
  • 👩‍💻 canary ブランチに対するプルリクエストである
  • zenn-cli で実行して正しく動作しているか確認する
  • 不要なコードが含まれていないか( コメントやログの消し忘れに注意 )
  • XSS になるようなコードが含まれていないか
  • モバイル端末での表示が考慮されているか
  • Pull Request の内容は妥当か( 膨らみすぎてないか )

より詳しい内容は Pull Request Policy を参照してください。

@cm-wada-yusuke cm-wada-yusuke added the dependencies 依存関係のアップデート label Jul 23, 2024
@cm-igarashi-ryosuke cm-igarashi-ryosuke self-assigned this Aug 13, 2024
Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

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

@senkenn 確認が遅くなってすみません!PR大変ありがたいです!1点、細かいコメントをしましたのでご確認をお願いします。

package.json Outdated Show resolved Hide resolved
@cm-igarashi-ryosuke cm-igarashi-ryosuke removed their assignment Aug 13, 2024
@senkenn
Copy link
Contributor Author

senkenn commented Aug 13, 2024

@cm-igarashi-ryosuke 修正しました!

Copy link
Member

@cm-igarashi-ryosuke cm-igarashi-ryosuke left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございました!
LGTMですのでマージします!

@cm-igarashi-ryosuke cm-igarashi-ryosuke merged commit d70f6c1 into zenn-dev:canary Aug 13, 2024
3 checks passed
@senkenn senkenn deleted the chore/senkenn-pnpmv9 branch August 13, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies 依存関係のアップデート
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants